Review Request 119183: A HSV color slider docker.

[Wolthera van Hövell]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119183/
-----------------------------------------------------------

(Updated Sept. 15, 2014, 4:57 p.m.)


Status
------

This change has been marked as submitted.


Review request for Calligra, Dmitry Kazakov and Boudewijn Rempt.


Bugs: https://bugs.kde.org/show_bug.cgi?id=313787
    http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=313787


Repository: calligra


Description
-------

I have been working on this https://bugs.kde.org/show_bug.cgi?id=313787.

This patch adds a color sliders docker to Krita.
The intention is to eventually get to a docker that can can be modified through the preferences, so one could have only saturation sliders, or two saturations sliders and a value slider.
This docker is only intended for derived models, where the specific colour selector is for actual colour spaces.


Diffs
-----

  krita/plugins/extensions/dockers/CMakeLists.txt f94eb44 
  krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_settings.cpp f171c2e 
  krita/plugins/extensions/dockers/advancedcolorselector/wdg_color_selector_settings.ui dbe9070 
  krita/plugins/extensions/dockers/colorslider/CMakeLists.txt PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider.h PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider.cpp PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider_dock.h PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider_dock.cpp PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider_input.h PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider_input.cpp PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider_widget.h PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_color_slider_widget.cpp PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_hsv_slider.h PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/kis_hsv_slider.cpp PRE-CREATION 
  krita/plugins/extensions/dockers/colorslider/krita_colorslider.desktop PRE-CREATION 
  libs/pigment/KoColorConversions.cpp fc7954b 

Diff: https://git.reviewboard.kde.org/r/119183/diff/


Testing
-------

A lot, no user testing, there's still a handful of open bugs.
We have 12 sliders, for each of the HSX colour models.
It compiles, and it doesn't crash.
* ~~Kocolorsliders wasn't intended for HSX, so we'll need to write a new widget for HSX.~~[done]
* ~~For now, all 12 sliders show up using kocolor sliders. I have managed to prevent it from zealously updating everything else, so it doesn't break other things.~~ [Thanks to the fix by dmitry on the specific colour selector, this has been fixed]
* ~~The number-input is broken. Half of the time~~ [fixed, I made it so that it will only change after loosing focus]
* ~~So is the slider input itself, I suspect this is rounding errors again, and will have to adjust the self-update, like I did for the colour palette.~~ We need to increase the precision of all the algoritms.
* ~~Config still needs to be put in.[done, however, after the config is changed, it doesn't re-update the docker again, on restart the update takes place.]~~[done]
* ~~There's still licenses missing~~[done]
* ~~The code is a bloody mess.~
* ~~There's a bug in the hsy and hsi code I wrote, but there's also bugs in the qt hsv and hsl code(gray in particular is strange) I'll chase these down with increasing the precision.~~ Fixed.
* There's some awkwardness between the different colour models updating eachother, leading to slider-dancing. Hopefull this'll be fixed with precision increase.
* The sliders don't update properly on LUT changes. This also happens in the specific colour selector.
I want to do these last two in a seperate patch.
* Made the H and S sliders work when the colour is black, as requested by Sven.
* Aligned the labels to the right..
* Couldn't set the labels to align perfectly.


Thanks,

Wolthera van Hövell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140915/647b682e/attachment.htm>