[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KColor is coming this Monday...
From:       Michael Pyne <michael.pyne () kdemail ! net>
Date:       2007-05-26 1:28:41
Message-ID: 200705252128.44977.michael.pyne () kdemail ! net
[Download RAW message or body]

[Attachment #2 (multipart/mixed)]


On Friday 25 May 2007, Matthew Woehlke wrote:
> Michael Pyne wrote:
> > Zack's screenshot is pretty convincing [snip]
>
> Well I would like to thank Zack for amply demonstrating the importance
> of API documentation. :-) And for making this demo so much easier to write.

Code always helps. :)

> Zack's screenshot looks "convincing" because he /asked/ KColor to tinker
> with the saturation as well as the luminance. Incidentally this is a
> KColor feature that QColor lacks. (Yes, I acknowledge that he probably
> did this because KColor isn't well documented yet.)

You're right, I removed the third parameter and then QColor and KColor become 
identical.

> And for the grand finale, a demonstration of how QColor can't keep its
> hands off the saturation when you /don't/ want to play with it is
> attached. Note that the KColor bars do /not/ result in fully saturated
> colors when starting with a gray tone, whereas the QColor bars do. You
> can also note how the white point depends on the starting color with
> QColor, which is not true for KColor.

To help me visualize I added a slider that adjust the initial saturation of 
the four colors before drawing the bands.  I'm not sure what's going on with 
the calculator for QColor when val >= 100 but assuming that's correct it 
seems you're right about the amount of lightening to reach white being 
dependant on the color.

But I guess the question for kdelibs is, do applications need a blend() or do 
applications need the full-blown capabilities of KColor?  And if we put in 
KColor, who's going to add HSL support to kcolorchooser? ;)

Regards,
 - Michael Pyne

["demo3.tar.bz2" (application/x-tbz)]
["demo3.png" (image/png)]
["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]