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 764857 - [PATCH] Small theme improvements
[PATCH] Small theme improvements
Status: RESOLVED FIXED
Product: gnome-flashback
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Flashback Maintainers
GNOME Flashback Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-04-10 18:23 UTC by Balló György
Modified: 2016-04-25 20:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
theme/Adwaita: Improve contrast of the notification (1.13 KB, patch)
2016-04-10 18:27 UTC, Balló György
none Details | Review
theme/HighContrast: fix tray icon color (826 bytes, patch)
2016-04-10 18:27 UTC, Balló György
none Details | Review
theme/HighContrast: use regular icons (849 bytes, patch)
2016-04-10 18:55 UTC, Balló György
none Details | Review

Description Balló György 2016-04-10 18:23:30 UTC
1. theme/Adwaita: Improve contrast of the notification bubbles

This makes the text more readable.


2. theme/HighContrast: fix tray icon color

HighContrast theme uses symbolic icons, but tray icons are invisible without setting a color for the images.
Comment 1 Balló György 2016-04-10 18:27:10 UTC
Created attachment 325675 [details] [review]
theme/Adwaita: Improve contrast of the notification
Comment 2 Balló György 2016-04-10 18:27:54 UTC
Created attachment 325676 [details] [review]
theme/HighContrast: fix tray icon color
Comment 3 Balló György 2016-04-10 18:55:09 UTC
Created attachment 325678 [details] [review]
theme/HighContrast: use regular icons

Unfortunately the second patch works only if I set the style from the inspector. I think the color is probably inherited from panel's system tray, but the opacity is configured to something random value. Maybe the alpha value is uninitialised in the panel?
Comment 4 Balló György 2016-04-10 19:13:34 UTC
It looks like the alpha value is uninitialized here, so we can't use symbolic icons until it's not fixed:
https://git.gnome.org/browse/gtk+/tree/gtk/deprecated/gtktrayicon-x11.c#n550
Comment 5 Balló György 2016-04-10 20:00:21 UTC
I sent a patch for the gtk+ project, which fixes the problem with symbolic tray icons:
https://bugzilla.gnome.org/show_bug.cgi?id=764863

Once it's merged, my second patch is no longer needed.
Comment 6 Alberts Muktupāvels 2016-04-25 20:01:17 UTC
It looks very readable to me... Do we really need black text?
Comment 7 Balló György 2016-04-25 20:20:10 UTC
Now it looks good without my patch... I think that something is changed between gtk3 3.20.2 and 3.20.3.

So I think that none of these patches are needed now. :)
Comment 8 Alberts Muktupāvels 2016-04-25 20:30:22 UTC
Ok, then I will close this bug as fixed.