Fix color role in services list

Review Request #112483 - Created Sept. 3, 2013 and submitted

Information
Christoph Feck
kde-baseapps
KDE/4.11
286459
Reviewers
dolphin
The service list uses Base background, so it requires Text color role. CheckBoxes use WindowText color role, because they are made for Window background.

Instead of "hacking" the palette, a better fix would be to not use CheckBox, but render the label separately. I tried that, but found issues with selection/focus rectangle, and settled for the hack. We use palette hacks at a few places in kdelibs, too.

Oh, forget to add:
This does not change the text color for selected items, this would indeed require a separate label text.

  
Christoph Feck
Frank Reininghaus
Commit Hook
Christoph Feck
Review request changed

Status: Closed (submitted)

Loading...