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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200203/aab41b38/attachment-0001.html>