Replace Adjustment and Effects menu icons with GTK icons#2102
Replace Adjustment and Effects menu icons with GTK icons#2102yioannides wants to merge 16 commits intoPintaProject:masterfrom
Adjustment and Effects menu icons with GTK icons#2102Conversation
Updated attribution formatting and added GIMP icons source.
There was a problem hiding this comment.
And this one is used for the Feather Object effect
There was a problem hiding this comment.
I see yeah, we can merge them for now and make sure I include two new icons for Feather Object and Brightness/Contrast in the next PR, as well as the icon reference in the code.
There was a problem hiding this comment.
So sorry for the delay, how does it look now? It took me a very long time to find where the effect names were declared and didn't realize some weren't even declared (this is my first time contributing code on this project, sorry for my messy commit pattern)
There was a problem hiding this comment.
This one is used for Brightness/Contrast
There was a problem hiding this comment.
This one's coincidentally fine, but I'd prefer using GNOME's sun symbolic for brightness and most of all, avoid icon use overlap at all costs.
|
Phew, so sorry for my messy commits, I'm still learning! I added all the new icons in a new |
|
I can do some debugging of this later, but there were some issues with icons either not loading, or not being re-colored, when I did a quick test on macOS 
|
|
At least the new |
3 participants
As previously discussed in #1428 (comment), it would be good to eventually replace all custom icons with icons that follow the same Adwaita design language and guidelines, either by the official team or the community.
The
adjustments-brightnesscontrast-symbolic.svgcomes from Adwaita's icon library andeffects-default-symbolic.svgcomes from GIMP's icon repo and chosen for its ubiquity in the GNOME/FOSS community, as previously discussed (I double-checked the icon license and we're all clear).