Bug 305377

Summary: Too easy to add new points in Adjust Curves
Product: [Applications] digikam Reporter: Milan Zamazal <pdm>
Component: Plugin-Editor-CurvesAssignee: Digikam Developers <digikam-bugs-null>
Status: REPORTED ---    
Severity: wishlist CC: caulier.gilles
Priority: NOR    
Version: 2.6.0   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Milan Zamazal 2012-08-18 09:46:27 UTC
When adjusting curves in the image editor it's easy (especially with some input devices) to create new curve points ("red dots") instead of moving existing ones. It would be nice if a different command, e.g. right clicking the mouse, was used to create new points.

Reproducible: Always

Steps to Reproduce:
1. Open Adjust Curves tool in image editor.
2. Adjust the curve in some way.
3. Click a bit far from some of the adjustment points.
Actual Results:  
A new curve point is created and I have to wait for redraw. Then I have to move the mistaken point away (hopefully clicking right this time) and wait again for another redraw. Then I can continue with my original intentions.

Expected Results:  
Either nothing happens or the closest point is moved. New points should be created with a different command, e.g. right mouse click.
Comment 1 Marcel Wiesweg 2012-09-23 14:05:19 UTC
Once again, I object to add barriers which disable unexperienced users. My suggestion: Extend the "move" region around existing points, and change the cursor shape to be explicit and give direct feedback about the action at the current position.
Comment 2 Milan Zamazal 2012-09-23 17:09:41 UTC
Extending the move region wouldn't work here (while it would be useful in the aspect ratio crop tool) since the curve adjustment widget is too small and this would prevent adding new points nearby (there is no such problem in aspect ratio crop).

The cursor shape already changes but it feels inexact and it's too easy to slip on click and performing the other action than originally indicated by the cursor.