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 work 
in //linear// color space. (I expect this is the reason `KHCY` and `QColor` 
produce different answers.)

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

Reply via email to