From b0c79a8bf41d3ae79c23e53afdcdba92ec0ec0b5 Mon Sep 17 00:00:00 2001 From: Nils Jeisecke Date: Tue, 10 Apr 2018 15:18:10 +0200 Subject: Flickable: fix another possible cause for out-of-sync moving flags In addition to d868bb4f3e4b0424fd4a2989ff1c82692b0f014c this removes the check for scrollingPhase in movementEnding. If movementEnding is invoked by some codepath other than timerEvent (e.g. setContentY) and scrollingPhase is true this will again prevent any further invocation of movementStarting from within the drag method (see d868bb4). As this check was introduced together with the movementEnding timer (QTBUG-63026) and scrollingPhase is now checked inside the timerEvent there should be no need for the check in movementEnding. Task-number: QTBUG-67460 Change-Id: I88ad6e3ee56b88a66bb61798b8876324f4842f1e Reviewed-by: Shawn Rutledge --- src/quick/items/qquickflickable.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/quick/items/qquickflickable.cpp b/src/quick/items/qquickflickable.cpp index 9c775f7e93..d11f6ea720 100644 --- a/src/quick/items/qquickflickable.cpp +++ b/src/quick/items/qquickflickable.cpp @@ -2678,15 +2678,13 @@ void QQuickFlickable::movementEnding(bool hMovementEnding, bool vMovementEnding) if (hMovementEnding && d->hData.moving && (!d->pressed && !d->stealMouse)) { d->hData.moving = false; - if (!d->scrollingPhase) - d->hMoved = false; + d->hMoved = false; emit movingHorizontallyChanged(); } if (vMovementEnding && d->vData.moving && (!d->pressed && !d->stealMouse)) { d->vData.moving = false; - if (!d->scrollingPhase) - d->vMoved = false; + d->vMoved = false; emit movingVerticallyChanged(); } if (wasMoving && !isMoving()) { -- cgit v1.2.3