Review Request: Improve KColorChooser dialog color preset selection implementing a custom ItemDelegate

Darío Andrés andresbajotierra at
Sun Mar 28 23:35:17 BST 2010

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-03-28 22:35:17.233429)

Review request for kdelibs and usability.


Added Aurélien mockups


Do not use the QPalette::Highlight color to show the selected color preset in the KColorChooser dialog. 

Use a custom selection drawing, black or white (depending on the contrast with the original color (KColorUtils::constrastRatio > 3.5))
This way you get a "this color is selected" hint, but you are still able to see which color it was/is

My last choice was a centered empty circle. Drawing a rectangle (or dotted rectangle) had too much glitches as the color preset buttons in the tableview are small (~ 13x13 pixels)
Also, there is a bug (not caused by me) that reduces the color preset buttons height or width (by one) on the top/left borders (seems to be related to the QTableWidget without Headers..)

I tried to not use color shading as it is still confusing to the user...

As a gift, I added a small detail in the empty buttons (no color defined), drawing a cross on it (using the background color)

Attached screenshot: can you recognize the selected color in the first one ? (hint: the same color is selected in both screenies..)

This addresses bug 230524.


  svn:// 1107102 



It works Ok and it looks good (ok, it could be prettier, any ideas? may be a gradient?)
At least it doesn't block the real color.


Before and After my patch
With a radiobutton-like selection
Using sunken border on selected item
Raised and plain
Raised and sunken
Plain and raised
Aurélien's idea 1
Aurélien's idea 2
Aurélien's idea 3



More information about the kde-core-devel mailing list