From 0c3866a5ba406fe7d04ec19cbf8378e9b0150fb0 Mon Sep 17 00:00:00 2001 From: Shawn Rutledge Date: Thu, 14 Jan 2021 18:27:14 +0100 Subject: Flickable: ignore trackpad events with px deltas in disallowed direction If Flickable.flickDirection == HorizontalFlick, then if the accumulated QWheelEvent::pixelDelta()'s abs(dx) > 2 * abs(dy), clearly the user is trying to scroll horizontally; otherwise, don't accept the event. That way the event is allowed to propagate to a parent Flickable that does allow flicking vertically. Likewise if the nesting is the other way around, only allow the inner vertical Flickable to accept if the flicking is actually vertical. Fixes: QTBUG-57245 Fixes: QTBUG-80236 Change-Id: Ieb0bf9310a67210ce7e9fe7a80c88baef2cc7ede Reviewed-by: Volker Hilsheimer (cherry picked from commit dccd8f0b5ca8f6faefb49718e33f9090243f3202) Reviewed-by: Qt Cherry-pick Bot --- .../quick/qquickflickable/tst_qquickflickable.cpp | 61 ++++++++++++++++++++++ 1 file changed, 61 insertions(+) (limited to 'tests/auto/quick/qquickflickable/tst_qquickflickable.cpp') diff --git a/tests/auto/quick/qquickflickable/tst_qquickflickable.cpp b/tests/auto/quick/qquickflickable/tst_qquickflickable.cpp index 2bd5d6bd1b..b7a1848949 100644 --- a/tests/auto/quick/qquickflickable/tst_qquickflickable.cpp +++ b/tests/auto/quick/qquickflickable/tst_qquickflickable.cpp @@ -170,6 +170,7 @@ private slots: void returnToBounds_data(); void wheel(); void trackpad(); + void nestedTrackpad(); void movingAndFlicking(); void movingAndFlicking_data(); void movingAndDragging(); @@ -959,6 +960,66 @@ void tst_qquickflickable::trackpad() QCOMPARE(flick->property("movementsAfterEnd").value(), 0); // QTBUG-55886 } +void tst_qquickflickable::nestedTrackpad() +{ + QQuickView window; + QVERIFY(QQuickTest::showView(window, testFileUrl("nested.qml"))); + + QQuickFlickable *rootFlickable = qmlobject_cast(window.rootObject()); + QVERIFY(rootFlickable); + QQuickFlickable *innerFlickable = rootFlickable->findChild(); + QVERIFY(innerFlickable); + QSignalSpy outerMoveEndSpy(rootFlickable, SIGNAL(movementEnded())); + QSignalSpy innerMoveEndSpy(innerFlickable, SIGNAL(movementEnded())); + QPoint pos = innerFlickable->mapToScene(QPoint(50, 50)).toPoint(); + quint64 timestamp = 10; + + // Scroll horizontally + for (int i = 0; i < 10 && qFuzzyIsNull(innerFlickable->contentX()); ++i) { + QWheelEvent event(pos, window.mapToGlobal(pos), QPoint(-10,4), QPoint(-20,8), + Qt::NoButton, Qt::NoModifier, i ? Qt::ScrollUpdate : Qt::ScrollBegin, false, + Qt::MouseEventSynthesizedBySystem); + event.setAccepted(false); + event.setTimestamp(timestamp++); + QGuiApplication::sendEvent(&window, &event); + } + QVERIFY(innerFlickable->contentX() > 0); + QCOMPARE(innerFlickable->contentY(), qreal(0)); + { + QWheelEvent event(pos, window.mapToGlobal(pos), QPoint(0,0), QPoint(0,0), + Qt::NoButton, Qt::NoModifier, Qt::ScrollEnd, false, + Qt::MouseEventSynthesizedBySystem); + event.setAccepted(false); + event.setTimestamp(timestamp++); + QGuiApplication::sendEvent(&window, &event); + } + QTRY_COMPARE(innerMoveEndSpy.count(), 1); + + innerFlickable->setContentX(0); + QCOMPARE(innerFlickable->contentX(), qreal(0)); + + // Scroll vertically + for (int i = 0; i < 10 && qFuzzyIsNull(innerFlickable->contentY()); ++i) { + QWheelEvent event(pos, window.mapToGlobal(pos), QPoint(4,-10), QPoint(8,-20), + Qt::NoButton, Qt::NoModifier, i ? Qt::ScrollUpdate : Qt::ScrollBegin, false, + Qt::MouseEventSynthesizedBySystem); + event.setAccepted(false); + event.setTimestamp(timestamp++); + QGuiApplication::sendEvent(&window, &event); + } + QVERIFY(rootFlickable->contentY() > 0); + QCOMPARE(rootFlickable->contentX(), qreal(0)); + { + QWheelEvent event(pos, window.mapToGlobal(pos), QPoint(0,0), QPoint(0,0), + Qt::NoButton, Qt::NoModifier, Qt::ScrollEnd, false, + Qt::MouseEventSynthesizedBySystem); + event.setAccepted(false); + event.setTimestamp(timestamp++); + QGuiApplication::sendEvent(&window, &event); + } + QTRY_COMPARE(outerMoveEndSpy.count(), 1); +} + void tst_qquickflickable::movingAndFlicking_data() { QTest::addColumn("verticalEnabled"); -- cgit v1.2.3