269904 2011-04-01 23:42:09 +0000 Checkboxes have wrong colors in ccsm main page 2011-07-29 23:28:34 +0000 1 1 4 Plasma Oxygen gtk2-engine unspecified Unlisted Binaries Linux CLOSED FIXED NOR normal --- 0 b7.10110111 hugo.pereira.da.costa b7.10110111 hugo.pereira.da.costa web 0 oldest_to_newest 1103277 0 b7.10110111 2011-04-01 23:42:09 +0000 Version: unspecified OS: Linux Reproducible: Always Steps to Reproduce: 1. Start ccsm, observe checkboxes being darker from top to bottom (blending with default oxygen background gradient). 2. Scroll to the end, see checkboxes at the top being dark 3. Hover any of them - they become white. Expected Results: I think blending should be disabled for cases when parent has custom background set (so that all the checkboxes in ccsm would be white). 1103332 1 hugo.pereira.da.costa 2011-04-02 10:48:28 +0000 I agree with the suggestion but it is actually hard to implement. There is already a check on whether checkbox is in list view, which fails for ccsm (cause they are actually not using a list view but some custom widget). Now the problem is: the said parent is usually not the direct parent of the checkbox that is being repainted (you can have various layouts in between, for instance, e.g. GtkBox). So should we check the custom background for *all* parents, recursively ? I fear some overhead. (especially since, if we want to do it consistently, it needs to be done for all widgets that we blend one way or the other). What do you think ? 1103346 2 hugo 2011-04-02 11:40:12 +0000 Git commit 78fdd3b958c9ec0642e83c79b6bdea046771a97a by Hugo Pereira Da Costa. Committed on 02/04/2011 at 11:40. Pushed by hpereiradacosta into branch 'master'. added recursive Gtk::gtk_widget_has_custom_background method. CCBUG: 269904 M +12 -0 src/oxygengtkutils.cpp M +3 -0 src/oxygengtkutils.h http://commits.kde.org/oxygen-gtk/78fdd3b958c9ec0642e83c79b6bdea046771a97a 1103347 3 hugo 2011-04-02 11:40:12 +0000 Git commit 1d2d6692dbcfb1cd5e5a89c17e97641b16e9a19d by Hugo Pereira Da Costa. Committed on 02/04/2011 at 11:41. Pushed by hpereiradacosta into branch 'master'. use Gtk::gtk_widget_has_custom_background to decide whether checkboxes and radiobuttons should be blended. CCBUG: 269904 M +6 -3 src/oxygenstylewrapper.cpp http://commits.kde.org/oxygen-gtk/1d2d6692dbcfb1cd5e5a89c17e97641b16e9a19d 1103349 4 hugo.pereira.da.costa 2011-04-02 11:41:40 +0000 ok. That fixes it. So far I have implemented extra check only for checkboxes and radiobuttons. It is doable for normal buttons but painful, (because drawButtonSlab is used all over the place, notably due to our custom comboboxes), so I'll wait for bug reports to show up :)