[Bug 246772] New: Curves: Don't recalculate on mouseUp if the position of the marker did not change

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 246772] New: Curves: Don't recalculate on mouseUp if the position of the marker did not change

Simon-2
https://bugs.kde.org/show_bug.cgi?id=246772

           Summary: Curves: Don't recalculate on mouseUp if the position
                    of the marker did not change
           Product: digikam
           Version: 1.4.0
          Platform: Compiled Sources
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: Image Editor
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:           1.4.0 (using KDE 4.4.5)
OS:                Linux

Related to: https://bugs.kde.org/show_bug.cgi?id=238776

If I drag a node around until it fits and then release the mosue button, it is
always recalculated. This is an unnecessary delay if the node's position did
not change.

Reproducible: Always




OS: Linux (x86_64) release 2.6.34-0.slh.11-sidux-amd64
Compiler: cc

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 246772] Curves: Don't recalculate on mouseUp if the position of the marker did not change

Gilles Caulier-4
https://bugs.kde.org/show_bug.cgi?id=246772


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
          Component|Image Editor                |Adjust Curves
            Product|digikam                     |digikamimageplugins




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 246772] Curves: Don't recalculate on mouseUp if the position of the marker did not change

Marcel Wiesweg
In reply to this post by Simon-2
https://bugs.kde.org/show_bug.cgi?id=246772


Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED




--- Comment #1 from Marcel Wiesweg <marcel wiesweg gmx de>  2011-01-27 19:44:25 ---
SVN commit 1217517 by mwiesweg:

No need to emit curvesChanged signal from mouseReleased.
It's sufficient to release it from mousePress and mouseMove.

Use update() instead of repaint().

BUG: 246772


 M  +4 -1      NEWS  
 M  +12 -14    libs/dimg/filters/curves/curveswidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1217517

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 246772] Curves: Don't recalculate on mouseUp if the position of the marker did not change

Gilles Caulier-4
In reply to this post by Simon-2
https://bugs.kde.org/show_bug.cgi?id=246772

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |2.0.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 246772] Curves: Don't recalculate on mouseUp if the position of the marker did not change

bugzilla_noreply
In reply to this post by Simon-2
https://bugs.kde.org/show_bug.cgi?id=246772

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|Adjust Curves               |Tool-Curves
            Product|digikamimageplugins         |digikam

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel