GNOME Bugzilla – Bug 751021
Use symbolic icons for fullscreen toolbar
Last modified: 2015-06-23 07:12:44 UTC
The headerbar uses symbolic icons now, but the migration has not been done for fullscreen toolbar yet. This patch solves the issue.
Created attachment 305352 [details] [review] Use symbolic icons for fullscreen toolbar
Created attachment 305697 [details] [review] Use symbolic icons for fullscreen toolbar
Created attachment 305698 [details] [review] Use symbolic icons for fullscreen toolbar
With the addition of the symbolic icon for the gallery, should I just remove the non-symbolic icon now? Apart from this detail, I think the patch is ready now so you can review it. :-)
Created attachment 305708 [details] [review] Remove non-symbolic gallery icon It is now useless since we use the symbolic one
I went ahead and removed the non symbolic icon. Feel free to reject this patch if you think we should keep it around.
Thanks! I also symbolized the leave-fullscreen icon, you seem to have forgotten. Well see if the new symbolized gallery icon is expressive enough, as it's not that clear that it's four images. But I tried symbolizing the previous icon by using Adwaita's generic image icon and that didn't look any better. ;) --- This problem has been fixed in the unstable development version. The fix will be available in the next major software release. You may need to upgrade your Linux distribution to obtain that newer version.
(In reply to Felix Riemann from comment #7) > I also symbolized the leave-fullscreen icon, you seem to have forgotten. Ah! It's not that I forgot, I didn't seem to find it. I probably didn't look for it hard enough. ;-) Thanks for the quick review.