search

elementary / icons

Named, vector icons for elementary OS
https://elementary.io
GNU General Public License v3.0
550 stars 84 forks source link

Adding color to cursors #1248

Closed wpkelso closed 7 months ago

wpkelso commented 8 months ago

This does some basic work towards addressing #1065 , starting with the watch animation in both its standalone and ptr form. I liked nakotami's idea with the pencil coloring from here, so that has been added in as a starting point; other "real" cursors will eventually follow suit (zoom-in/zoom-out).

--EDIT--

This PR now encompasses not just the work addressing #1065, but also small modifications to many other cursor files to bring them more in-line with the rest of the set in addition to clarity improvements.

wpkelso commented 8 months ago

Current status of things

I tried not to touch the main bodies of cursors where possible, as well as keep similar icons on similar color-lines. I think it might be a good idea to keep the all-black set of cursors as an option available to the user for accessibility purposes.

wpkelso commented 8 months ago

Not sure what happened with the USB changes, nothing actually changed there to my knowledge. However, I've reverted that file to main.

I reverted the watch cursors as well, I'll revisit them in the issue thread for a more thorough design exploration.

Changing to symbol-on-circle definitely helps with clarity for the dnd-cursors, although the size constraints meant I had to do some reworking of the symbols themselves.

wpkelso commented 7 months ago

@danirabbit I think the original badges I was working with here were based on an earlier version of the HIG versus what you've done in #1250 , and I just failed to realize it.

I'll go ahead and split this up and close this PR. Hopefully that will resolve the issue with the USB receiver icon (I thought it had been unified with what was on main but perhaps that got pulled in before I had added the precommit on my end and thus wasn't cleaned up on main).