Prevent timer-triggered property update when value is already committed

Stop slider rate-limiting timer when user releases mouse button and value
is committed to property system, to prevent timer triggering an unnecessary
non-committing property change. Instead at button release just update the
associated text field to reflect the committed value.

Change-Id: Ia0587b1fa1c3860b0d07bd653d91e7dc2240c293
Reviewed-by: Mahmoud Badri <mahmoud.badri@qt.io>
Reviewed-by: Marianne Yrjänä <marianne.yrjana@qt.io>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>

1 files changed, 5 insertions, 1 deletions

diff --git a/src/Authoring/Studio/Palettes/Action/HandlerPropertyBaseSlider.qml b/src/Authoring/Studio/Palettes/Action/HandlerPropertyBaseSlider.qml
index ad016b86..853ed438 100644
--- a/src/Authoring/Studio/Palettes/Action/HandlerPropertyBaseSlider.qml
+++ b/src/Authoring/Studio/Palettes/Action/HandlerPropertyBaseSlider.qml
@@ -117,8 +117,12 @@ Row {
         }
 
         onPressedChanged: {
-            if (!pressed)
+            if (!pressed) {
+                if (rateLimiter.running)
+                    rateLimiter.stop();
+                textField.setTextFieldValue()
                 root.editingFinished();
+            }
         }
 
         MouseArea {