Version: 0.6.2 (using KDE 4.6.2) OS: Linux Many functions have R, G, B, Alpha and Luminosity channels that can be adjusted independently. The "Reset" button should reset all of the channels and not just the visible item. Reproducible: Always Steps to Reproduce: Working with an image, for instance Adjusting Curves, change any 2 channels. Now save the result and exit. When one now wishes to use the same function the normal response is to reset the visible color ... the second color remains locked as previously used. Actual Results: This easily results in odd color casts being applied inadvertently to images since settings for all functions remain in place from one image to the next. Expected Results: With all color functions I expect that pressing the 'reset' button will clear all channels to the neutral state. OS: Linux (i686) release 2.6.35-30-generic-pae Compiler: cc
Do you use 0.6.2 version seriously ? Gilles Caulier
Version Statement Error: Should read 2.0.0-RC
I stumbled upon this too recently. Do we agree the behavior should be changed as described? There is no reset button then for a single curve.
Just came across this today. It took me a while until I realised why clicking on "reset" would not show me the original as "after"-version. So I would agree that "reset" should reset everything. Maybe one could add the tiny reset buttons the luminosity curve in the demosaicing tool has at its bottom-left and make that one only reset the current curve.
(In reply to comment #3) > I stumbled upon this too recently. Do we agree the behavior should be changed > as described? There is no reset button then for a single curve. IMHO yes, there should be a way to reset all the curves with one click, may be better to wait for Gilles tought > Actual Results: > This easily results in odd color casts being applied inadvertently to images > since settings for all functions remain in place from one image to the next. meetoo ;-)
look on thebottom left corner of curve widet, there is a mall button to reset curve. gilles caulier
Hm, dont see it... There's the big button on the right, which is the point of discussion here. The five small buttons on the left do not reset: http://wstaw.org/m/2011/07/18/plasma-desktopX23365.jpg
Asking again for opinions: Anyone arguing that the Reset button should only reset the current channel, or that we need two separate Reset buttons, one only resetting the current channel, the other all channels?
Marcel, Why not to replace Reset button by a button pop-up menu with these actions : "Reset current channel" and "Reset all channels" ? Gilles Caulier
Created attachment 118515 [details] Adds Popup menu with reset channel and reset all actions.
Git commit 9c43a9ebd3b7db21a683bce72552b09685a241d0 by Maik Qualmann. Committed on 03/03/2019 at 21:50. Pushed by mqualmann into branch 'development/dplugins'. apply patch #118515 from Sambhav Dusad to add Popup menu with reset all action FIXED-IN: 6.1.0 M +29 -11 core/libs/dimg/filters/curves/curvesbox.cpp https://commits.kde.org/digikam/9c43a9ebd3b7db21a683bce72552b09685a241d0
*** Bug 308243 has been marked as a duplicate of this bug. ***