mwoehlke added inline comments.
INLINE COMMENTS
> kcolorutils.cpp:60
> *c = khcy.c;
> *h = khcy.h;
> *y = khcy.y;
I guess it will be less confusing if this is also normalized?
*h = khcy.h + (khcy.h < 0.0 ? 1.0 : 0.0);
> kcolorutils.h:36
> +/**
> + * Calculate the hue of a colo
mwoehlke added inline comments.
INLINE COMMENTS
> broulik wrote in kcolorutils.h:42
> This doesn't mention hcy anywhere and there's `QColor::hue`.
> But we already have `luma` in the same way, so I guess this is alright.
I expect it wouldn't hurt to mention that `luma`, `chroma` and `hue` all w
mwoehlke added a comment.
Indeed, we could have been having this conversation here instead of via
e-mail 😉. But, six of one...
I believe `luma` and `chroma` are already guaranteed to return normalized
values; it's just `hue` that you may want to normalize if you don't want to
"surprise"
* still remove
tabs? (Seems like you'd want it too in that case...)
- Matthew Woehlke
On April 22, 2014, 5:06 p.m., Sven Brauch wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.revi
On 2013-06-18 11:54, Aleix Pol wrote:
On Tue, Jun 18, 2013 at 5:40 PM, Matthew Woehlke wrote:
On 2013-06-18 03:10, Dominik Haumann wrote:
On Tuesday, June 18, 2013 08:14:50 Kevin Ottens wrote:
On Monday 17 June 2013 19:43:33 Aleix Pol wrote:
So I was looking at KGlobalSettings to see what we
On 2013-06-18 03:10, Dominik Haumann wrote:
On Tuesday, June 18, 2013 08:14:50 Kevin Ottens wrote:
On Monday 17 June 2013 19:43:33 Aleix Pol wrote:
So I was looking at KGlobalSettings to see what we need to do to finally
deprecate all KGlobalSettings, as suggested by the required kdelibs
cleanu
On 2012-08-18 03:50, David Faure wrote:
On Thursday 09 August 2012 17:16:27 Matthew Woehlke wrote:
I think you're going to have to either:
a. Accept a SIC and just let some roles be wrong for old code
b. Have a suffix on all the new flags
c. Put the new flags in some namespace that
On 2012-08-09 07:43, David Faure wrote:
On Friday 20 July 2012 14:19:47 Matthew Woehlke wrote:
I think going with flags is probably a good idea; I would probably do
that by keeping the existing sets, roles and shades as distinct, but
allow combining them into one variable, e.g.:
QPalette
On 2012-07-20 12:09, David Faure wrote:
On Thursday 19 July 2012 22:18:36 Matthew Woehlke wrote:
because I don't have a KDE build at the moment :-(
Too bad, I wanted to ask for your help on something related...
I'm hoping https://bugs.kde.org/show_bug.cgi?id=303825 may finally
