Using Oxygen-gtk in fedora (oxygen-gtk2-1.2.5-1.fc17.x86_64) and guitarix (guitarix-0.22.3-1.fc17.x86_64) under KDE (kde-runtime-4.8.4-2.fc17.x86_64) I find Guitarix menus (gtk2 app) look like the attached screenshot. Guitarix has a number of themes which set the menu bar and menu drop down background colours and font colours, but it seems the menu drop-down background colour changes are never honoured and the menu bar background colours are only honoured when using qt-curve or clearlook rather than oxygen-gtk. The behaviour is okay in Gnome3. I'm afraid I'm not very familiar with the KDE infrastructure so I'm not 100% this is the right component to file against. Reproducible: Always Steps to Reproduce: 1. Start guitarix in KDE 2. Try the drop down menus. If you can read them try the Gx7-blues or Gx1-alloy skins 3. Actual Results: Can't read menus Expected Results: Can read menus
it is indeed oxygen gtk bug that we do not honour curstom colors for the background in several places, which results in the bug you mention. Will investigate I guess in the meanwhile you can use the "default" theme ? (In general I really don't like themed applications as this goes against all efforts of consistency across desktop ...)
In fact, I have almost the same menu look with QtCurve: http://imgur.com/k0PDx . The only difference is the menu bar, which is dark here. Not sure if we really want to follow app-set custom window background colors.
@Ruslan, I agree that not respecting the custom colors at all might be a solution, but then we must make it consistently, that is, not respect the foreground color change either. The fact that the current colors are "hybrid" is what causes the problem. Whatever we decide, there are changes involved on our side. In the end I'm not sure that it is more difficult to just honour the custom background (as we do for main window already), though this depends on how it is set by the application. I'll have a look Hugo
Thanks. Indeed QtCurve has the same problem, but clearlook does not. I don't really think it's critical to the app to theme the menu-bar, but the default skin is one of the affected ones (and oxygen-gtk is Fedora's default gtk on KDE theme), so the source of the problem isn't immediately obvious.
@Ian can you post a screenshot of how the applications (notably menus) look with clearlooks ? (to make sure I understand things right) Thanks in adv Hugo
Hi, picture with clearlooks: https://picasaweb.google.com/lh/photo/jap3Spa98MbNltjFJz9orOPFHzJ9cFFhbUJUYSJLZfM?feat=directlink That's also how it appears in gnome.
@Ruslan Patch at: http://pastebin.com/xUHHeZD1 Fixes the issue. It is rather simple, so what do you think ? Basically it overwrites our Palette settings with whatever "Style->..." contains. Hopefully this does not break other things, provided that our default (on-fly generated) gtkrc is not broken. Only drawback is that it slows down a bit color lookup at each rendering (though I have not seen any visible change) Now I must say, well, to me the app simply looks plain ugly, themed or not :( not sure it is worth the effort ... PS: the mandatory screenshot: http://wstaw.org/m/2012/07/26/plasma-desktopRX3774.png
Looks good enough. Slowdown should be negligible, so I think this can be pushed.
Git commit 0e5335e3cbc2ea0aadf9d16a794acae8094f467e by Hugo Pereira Da Costa. Committed on 26/07/2012 at 18:38. Pushed by hpereiradacosta into branch 'master'. oxygenstyle: use color provided by options when available when rendering menu background. oxygenstylewrapper: use color provided by style to render menubar and menu background M +1 -1 src/oxygenstyle.cpp M +10 -1 src/oxygenstylewrapper.cpp http://commits.kde.org/oxygen-gtk/0e5335e3cbc2ea0aadf9d16a794acae8094f467e
Git commit 2d1e8ce65b81404936cddfe669e3a9b8b03d59ad by Hugo Pereira Da Costa. Committed on 26/07/2012 at 18:38. Pushed by hpereiradacosta into branch '1.3'. oxygenstyle: use color provided by options when available when rendering menu background. oxygenstylewrapper: use color provided by style to render menubar and menu background M +1 -1 src/oxygenstyle.cpp M +10 -1 src/oxygenstylewrapper.cpp http://commits.kde.org/oxygen-gtk/2d1e8ce65b81404936cddfe669e3a9b8b03d59ad
ok. Committed and backported. Closing. Thanks for reporting !
Confirmed works here, thanks for the quick fix!