GNOME Bugzilla – Bug 612371
text in keyboard status applet does not respect changes of theme colours
Last modified: 2010-04-15 21:03:35 UTC
Created attachment 155700 [details] keyboard applet in the system tray Binary package hint: gnome-control-center The text in the keyboard preferences applet, which appears in the system tray when multiple keyboard layouts are present does not follow theme guidelines. Steps to reproduce: 1) Open keyboard preferences from the control center 2) Select layouts tab 3) Add more than one layouts 4) Observe the keyboard preferences applet in the system tray 5) Change gnome theme, and observe that the colour of the text in the keyboard preferences applet does not change. I have attached a screenshot showing the system tray with the keyboard indicator when the Ambiance theme is selected from Lucid Lynx Alpha 3.
There is a tricky question here. I am trying to figure out if there is a way to get notified about theme change without having any gtk widget. Unfortunately the status icon is not a widget:(
Fixed in git. Could you please check?
Thanks, the text style is now updated when the theme change occurs, however, it appears that the wrong style is being used. Currently, the GTK_TYPE_LABEL from using the default widget path style is being applied, which appears to not be the same style as what is being used by other panel objects like the clock and menu. This is apparent when comparing the colour of the text in themes such as Ambiance where this text should be light, and the label text should be dark. In themes such as Dark Room, and all of the lighter themes this bug is not apparent because the panel text is a similar colour to the label text.
Created attachment 155906 [details] [review] Patch to select panel widget style info Herewith a patch to deal with the above-mentioned issue. I'm not sure that this is the correct approach though, but based on my testing, it works.
By any chance, do you know what style is being used by the clock applet and the menu?
Oops, sorry, I did not see your comment. Will try it now.
Done. Even though I cannot see the difference. Perhaps my version of Ambiance, from gnome-looks, is different from yours.
Thanks. It's is also worth noting that the keyboard indicator is now also visible in gnome-shell, where previously it was invisible with themes where text would have been black.
Shouldn't this bug be reassigned to the libgnomekbd product? And is there any relation to bug 611875?
*** Bug 611875 has been marked as a duplicate of this bug. ***
Yes it is actually related to libgnomekbd. There is a complex situation here, related to the fact that notification area can only accomodate square icons. See the latest discussions in desktop-devel-list
I think it is ok now. Please check in git. If it is ok - 2.30.1 will have it
So does this supposedly fix the font size and anti-aliasing issue? If it is related to our latest discussion in bug 611875, then we should try to pull it into Lucid.
Yes, that should fix it all. But I was informed that my fix introduced funny effect with defaul lucid theme. I have to investigate it before recommending to lucid. Well, it is too late anyway - but may be 10.04.1 release would incorporate it.