After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 760536 - ship a mini icon theme for recolarable symbolic icon assets
ship a mini icon theme for recolarable symbolic icon assets
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
3.19.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2016-01-12 16:06 UTC by Lapo Calamandrei
Modified: 2016-01-14 22:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
adwaita-private mini icon theme (3.17 KB, application/x-bzip)
2016-01-12 16:06 UTC, Lapo Calamandrei
  Details
Quick and dirty patch using the said theme as assets in Adwaita (96.30 KB, patch)
2016-01-12 16:07 UTC, Lapo Calamandrei
none Details | Review
Screenshot of the assets in use (8.04 KB, image/png)
2016-01-12 16:09 UTC, Lapo Calamandrei
  Details
theme: radios & checkboxes as buttons (113.69 KB, patch)
2016-01-13 18:01 UTC, Jakub Steiner
none Details | Review
an initial implementation (16.01 KB, patch)
2016-01-14 03:51 UTC, Matthias Clasen
none Details | Review
icon theme: Add a way to create GtkIconInfos from files (4.28 KB, patch)
2016-01-14 18:52 UTC, Matthias Clasen
committed Details | Review
icon theme: Export recoloring function (3.23 KB, patch)
2016-01-14 18:52 UTC, Matthias Clasen
committed Details | Review
Add a way to load recolored icons in CSS (10.89 KB, patch)
2016-01-14 18:52 UTC, Matthias Clasen
committed Details | Review

Description Lapo Calamandrei 2016-01-12 16:06:17 UTC
Created attachment 318892 [details]
adwaita-private mini icon theme

I'm rewriting the way we do checks and radios in adwaita using the new wonders available in 3.19, trying to avoid the use of a whole bunch of assets, which are a pain to keep up to date. I still need anyway 3 recolarable assets, the only way I have atm to achieve that is by using symbolic icons, so I created a mini theme of 3 symbolic icons. I wouldn't like much to ship those in adwaita-icon-theme since the size is pretty much custom (14x14) and it doesn't sound right to me to rely on external stuff for pretty specific adwaita needs.
I would like to know if it makes sense to you and if it's feasible to hide it in the resources.
In case the idea is RightTM, I'd need someone to take care of the build system work.
Comment 1 Lapo Calamandrei 2016-01-12 16:07:23 UTC
Created attachment 318893 [details] [review]
Quick and dirty patch using the said theme as assets in Adwaita
Comment 2 Lapo Calamandrei 2016-01-12 16:09:16 UTC
Created attachment 318894 [details]
Screenshot of the assets in use
Comment 3 Matthias Clasen 2016-01-13 03:06:09 UTC
I don't see an easy way to make a resource-only private icon theme, so we may have to go back to using resources for these 3 images.
Comment 4 Jakub Steiner 2016-01-13 18:01:18 UTC
Created attachment 318980 [details] [review]
theme: radios & checkboxes as buttons

- use assets for overlay checkmarks and radio pills
- non recolorable / doesn't work for dark
Comment 5 Matthias Clasen 2016-01-13 19:36:33 UTC
Here is an alternative suggestion that we can implement: Extend the <image> syntax to include

<recolorable image> = -gtk-recolor(<image>[,<palette>])

which would do the 'symbolic' thing to <image>, using the colors from <palette>, which would default to the value of -gtk-icon-palette.
Comment 6 Lapo Calamandrei 2016-01-13 19:56:13 UTC
-gtk-recolor() would do the trick allowing the use of normal assets.
Comment 7 Matthias Clasen 2016-01-14 03:51:16 UTC
Created attachment 318994 [details] [review]
an initial implementation
Comment 8 Matthias Clasen 2016-01-14 18:52:49 UTC
Created attachment 319041 [details] [review]
icon theme: Add a way to create GtkIconInfos from files

This is sometimes needed, and calling into actual icon theme
code just for it is confusing - the resulting icon does not
depend on the icon theme at all.
Comment 9 Matthias Clasen 2016-01-14 18:52:53 UTC
Created attachment 319042 [details] [review]
icon theme: Export recoloring function

This will be used in subsequent commits.
Comment 10 Matthias Clasen 2016-01-14 18:52:58 UTC
Created attachment 319043 [details] [review]
Add a way to load recolored icons in CSS

This can come in handy to load images that are not in an icon theme,
but still benefit from symbolic recoloring.
Comment 11 Matthias Clasen 2016-01-14 19:03:44 UTC
Attachment 319041 [details] pushed as 7d717c8 - icon theme: Add a way to create GtkIconInfos from files
Attachment 319042 [details] pushed as 1b0ef71 - icon theme: Export recoloring function
Attachment 319043 [details] pushed as beec55a - Add a way to load recolored icons in CSS