D27017: [KColorUtils] Add hue(), chroma() and getHcyColor()

[Matthew Woehlke]

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" users with its "natural" range of ¯¹⁄₆ - ⁵⁄₆.

INLINE COMMENTS

> kcolorspaces.cpp:161
> +{
> +    return KHCY(color).h;
> +}

So... per discussions, wrapping this in, well, `wrap` 😉 might be reasonable.

REPOSITORY
  R273 KGuiAddons

REVISION DETAIL
  https://phabricator.kde.org/D27017

To: ndavis, #frameworks, dfaure
Cc: mwoehlke, broulik, kde-frameworks-devel, LeGast00n, GB_2, michaelh, 
ngraham, bruns