Bug 391934 - Color Sliders (HSV) - the Saturation slider does not update when picking a color that has Value 100
Summary: Color Sliders (HSV) - the Saturation slider does not update when picking a co...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Dockers (show other bugs)
Version: 4.0
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-16 14:23 UTC by trash368031371575
Modified: 2018-05-02 12:00 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
(Video example) (969.27 KB, video/mp4)
2018-03-17 12:30 UTC, mvowada
Details

Note You need to log in before you can comment on or make changes to this bug.
Description trash368031371575 2018-03-16 14:23:28 UTC
Steps to Reproduce:
1) Pick any color.
2) Use the color picker tool to pick a color that has value 100.
Comment 1 mvowada 2018-03-17 12:30:55 UTC
Created attachment 111458 [details]
(Video example)

Yes, I can confirm in Krita 4 - Ubuntu 14.04 as well (see video). 
Reminder - Open reports I found concerning the "Color Sliders" docker:

    Bug 352940 (various inaccuracies)
    Bug 379691 (changing values when switching between documents and back)
    Bug 385405 (sliders move irrationally with CMYK color model)
Comment 2 trash368031371575 2018-04-10 13:10:32 UTC
Can someone fix this please?

krita/plugins/dockers/colorslider/kis_color_slider_input.cpp
at line 533:

if (val==0 || val>0.999) {
    m_sat = sat_backup;
}

should be:

if (val==0) {
Comment 3 Halla Rempt 2018-04-28 11:13:50 UTC
Git commit 2e8cf3911ed8dda1884a9991d77b5474be1ca1ca by Boudewijn Rempt.
Committed on 28/04/2018 at 11:12.
Pushed by rempt into branch 'master'.

Make the Saturation slider update when picking a color that has Value 100

M  +1    -1    plugins/dockers/colorslider/kis_color_slider_input.cpp

https://commits.kde.org/krita/2e8cf3911ed8dda1884a9991d77b5474be1ca1ca
Comment 4 Halla Rempt 2018-04-28 11:13:59 UTC
Thanks! I've pushed the change.
Comment 5 Halla Rempt 2018-05-02 12:00:21 UTC
Git commit 6c080e287c014779dca0db974f769a53280916db by Boudewijn Rempt.
Committed on 02/05/2018 at 11:54.
Pushed by rempt into branch 'krita/4.0'.

Make the Saturation slider update when picking a color that has Value 100
(cherry picked from commit 2e8cf3911ed8dda1884a9991d77b5474be1ca1ca)
(cherry picked from commit 99ed6c81caa9241adc13f2f292b440ee2991567b)

M  +1    -1    plugins/dockers/colorslider/kis_color_slider_input.cpp

https://commits.kde.org/krita/6c080e287c014779dca0db974f769a53280916db