Fix AutoScroller

Use the timer properly, restarting the timer instead of calling
onTriggered handler directly.

Connect to Qt.inputMethod.animation property, and update auto scroll
when the animation ends.

Simplify signal Connection usage, remove duplicate Connection instances
and just target Qt.inputMethod uncoditionally.

Task-number: QTBUG-66294
Change-Id: Ie82f91cbc8ef84b4a453fa981f2e638bec190073
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>

1 files changed, 5 insertions, 8 deletions

diff --git a/examples/virtualkeyboard/basic/content/AutoScroller.qml b/examples/virtualkeyboard/basic/content/AutoScroller.qml
index d8d84a50..2ea8efb0 100644
--- a/examples/virtualkeyboard/basic/content/AutoScroller.qml
+++ b/examples/virtualkeyboard/basic/content/AutoScroller.qml
@@ -52,7 +52,7 @@ Item {
                 }
                 parent_ = parent_.parent
             }
-            delayedLoading.triggered()
+            delayedLoading.restart()
         }
     }
 
@@ -96,12 +96,9 @@ Item {
     }
     Connections {
         ignoreUnknownSignals: true
-        target: inputItem && !Qt.inputMethod.animating ? Qt.inputMethod : null
-        onKeyboardRectangleChanged: delayedLoading.triggered()
-    }
-    Connections {
-        ignoreUnknownSignals: true
-        target: inputItem && inputItem.activeFocus ? inputItem : null
-        onCursorRectangleChanged: delayedLoading.triggered()
+        target: Qt.inputMethod
+        onAnimatingChanged: if (inputItem && !Qt.inputMethod.animating) delayedLoading.restart()
+        onKeyboardRectangleChanged: if (inputItem) delayedLoading.restart()
+        onCursorRectangleChanged: if (inputItem && inputItem.activeFocus) delayedLoading.restart()
     }
 }