search

indiana-university / rivet-icons

Icons for Indiana University's Rivet Design System
https://rivet.iu.edu/icons-stickers/
BSD 3-Clause "New" or "Revised" License
5 stars 6 forks source link

Add sync icon #17

Closed anmaxey closed 5 years ago

anmaxey commented 5 years ago

sync icon design discussed during CoP and here is the icon implemented.

anmaxey commented 5 years ago

@levimcg please review.

anmaxey commented 5 years ago

done on windows and there were some issues within the icons.json. The / characters were replaced with \\. I manually corrected this.

basham commented 5 years ago

Yes, there was a debate about the name. In general, we felt sync was a slightly stronger name for the icon. refresh could be better if perhaps it was the redo icon with a longer tail, similar to what you see in Chrome and other browsers. But we're probably weary that such an icon would be too close in nature to the redo icon. sync seems like it could work for both cases, while being sufficiently distinct.

I would advocate for not making aliases. That could create a cascade of unnecessary work. Perhaps, the icons could be documented with some description to elaborate on best uses. So, for the sync icon, we could just mention that it could be used for data sync or refresh.