[KDE Usability] Review Request: Improve KColorChooser dialog color preset selection implementing a custom ItemDelegate
Aurélien Gâteau
agateau at kde.org
Sun Mar 28 20:56:56 BST 2010
> On 2010-03-27 17:26:15, Aurélien Gâteau wrote:
> > It's definitely more usable but as you said it can probably be made prettier. Maybe you could draw a dot similar to radio button dots? Since this selection is quite similar to the way radio buttons work, it could be interesting to have a similar look.
>
> Darío Andrés wrote:
> I tested circle + inner dot and it looks nicer..
> Were you suggesting about only using a centered dot ?
>
> Aurélien Gâteau wrote:
> Yes, I was thinking only centered dot, but actually I am not sure it would look nice. If I find some time I am going to play with some mockups.
Here are some mockups. None of them really strike me as better, though:
- http://imagebin.ca/view/MsFh5Od.html
- http://imagebin.ca/view/IJRvdN.html
- http://imagebin.ca/view/bZyjeTa.html
- Aurélien
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3410/#review4701
-----------------------------------------------------------
On 2010-03-28 17:10:25, Darío Andrés wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3410/
> -----------------------------------------------------------
>
> (Updated 2010-03-28 17:10:25)
>
>
> Review request for kdelibs and usability.
>
>
> Summary
> -------
>
> 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.
> https://bugs.kde.org/show_bug.cgi?id=230524
>
>
> Diffs
> -----
>
> svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/kdeui/colors/kcolordialog.cpp 1107102
>
> Diff: http://reviewboard.kde.org/r/3410/diff
>
>
> Testing
> -------
>
> 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.
>
>
> Screenshots
> -----------
>
> Before and After my patch
> http://reviewboard.kde.org/r/3410/s/340/
> With a radiobutton-like selection
> http://reviewboard.kde.org/r/3410/s/341/
> Using sunken border on selected item
> http://reviewboard.kde.org/r/3410/s/345/
>
>
> Thanks,
>
> Darío
>
>
_______________________________________________
kde-usability mailing list
kde-usability at kde.org
https://mail.kde.org/mailman/listinfo/kde-usability
More information about the kde-core-devel
mailing list