Summary: | Incomplete Reset of Functions for Curves widget [patch] | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | David Vincent-Jones <davidvincentjones> |
Component: | Plugin-Editor-Curves | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | caulier.gilles, davidvincentjones, marcel.wiesweg, sambhavdusad24, sven.burmeister, vivo75+kde |
Priority: | NOR | Keywords: | junior-jobs |
Version: | 2.0.0 | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/digikam/9c43a9ebd3b7db21a683bce72552b09685a241d0 | Version Fixed In: | 6.1.0 |
Attachments: | Adds Popup menu with reset channel and reset all actions. |
Description
David Vincent-Jones
2011-07-04 02:08:00 UTC
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. *** |