Review Request 120291: cleaning up Advanced Colour Selector settings+Configurable HSY' luma.

[Wolthera van Hövell]

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

Review request for Calligra, Dmitry Kazakov, Sven Langkamp, and Boudewijn Rempt.


Repository: calligra


Description
-------

This patch tries to clean up the big list of colour selectors left by the last patch.
It contains a set of radio buttons that set the model type, and the list of widgets underneath is updated to only present colour selectors in that type.

It also adds some explaination of each type.

Beneath that, there's the Luma settings, with explaination. These values are saved to the kritarc file under advanced colour selector.


Diffs
-----

  krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_combo_box.h 486434a 
  krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_combo_box.cpp c6658a1 
  krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_settings.h d4240fe 
  krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_settings.cpp bbed40e 
  krita/plugins/extensions/dockers/advancedcolorselector/wdg_color_selector_settings.ui f83e768 

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


Testing
-------

It works but due to the widget list being reorganised, you need to select a selector first, save the preferences, to end up with the proper list. I am not sure why.

Todo:

* Make the luma-coefficients actually have effect.
* Have the information on the models be aligned properly.
* Make invisible widgets unselectable
* Maybe remove triangle from all non-hsv types: On one hand it is a true wildcard selector; it fits in all types, on the other hand due QGridLayout being pretty inflexible in regards to auto-filling it now looks ugly.
* Add missing hsi and hsy selectors.
* Organise the widgets more nicely.


Thanks,

Wolthera van Hövell

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