----------------------------------------------------------- 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
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
