From 262d7eb305e1dea8dac660bec3ccc50193258ea9 Mon Sep 17 00:00:00 2001 From: Shawn Rutledge Date: Wed, 11 Nov 2020 21:29:11 +0100 Subject: Fix QQWinPriv::deliverSinglePointEventUntilAccepted for wheel, gestures WheelHandler was only reacting to one wheel event between mouse moves, because it got added to the QQPointerHandlerPriv::deviceDeliveryTargets() vector, and was not removed at the beginning of delivery of subsequent events, as QQuickWindowPrivate::deliverPointerEvent() does. (In Qt 5 the equivalent vector was cleared in QQuickPointerMouseEvent::reset().) Wheel events are delivered via deliverSinglePointEventUntilAccepted() (grabbing the wheel is still not implemented). Native gesture events are delivered that way too; and sure enough, the same bug happens on the macOS trackpad, whether you are attempting to do pinch zoom or just two-finger-flick. tst_QQuickWheelHandler::nestedHandler() sends multiple wheel events in a row, so we do have some test coverage, and hopefully this issue explains why it needed to be blacklisted. Fixes: QTBUG-88428 Task-number: QTBUG-86729 Change-Id: Id1ed4a38dfa3eb2253c4a60f09f80aea0f69707e Reviewed-by: Volker Hilsheimer --- tests/auto/quick/pointerhandlers/qquickwheelhandler/BLACKLIST | 3 --- 1 file changed, 3 deletions(-) (limited to 'tests/auto/quick/pointerhandlers') diff --git a/tests/auto/quick/pointerhandlers/qquickwheelhandler/BLACKLIST b/tests/auto/quick/pointerhandlers/qquickwheelhandler/BLACKLIST index a189ebe041..2949d3371f 100644 --- a/tests/auto/quick/pointerhandlers/qquickwheelhandler/BLACKLIST +++ b/tests/auto/quick/pointerhandlers/qquickwheelhandler/BLACKLIST @@ -1,6 +1,3 @@ # QTBUG-81993 [singleHandler] macos ci -* # QTBUG-86729 -[nestedHandler] -* # QTBUG-86729 -- cgit v1.2.3