diff options
author | Shawn Rutledge <shawn.rutledge@qt.io> | 2020-10-14 12:39:38 +0200 |
---|---|---|
committer | Shawn Rutledge <shawn.rutledge@qt.io> | 2020-10-22 04:54:17 +0000 |
commit | 0e5bbf3507666954fefa5e3c4a2eea531ce71259 (patch) | |
tree | a14c074b9ddee8a2be4183e912d4563dcd3c34bc | |
parent | 3310e13a17d2249a86fa533e350744c5593be54f (diff) |
Make QEvent::setAccepted() virtual; set QEventPoints state the same
In Qt Quick, when we deliver an item-specific QTouchEvent that contains
only the subset of eventpoints that are inside the Item's bounds,
traditionally the Item can accept the event to tell the delivery logic
that the event is handled and doesn't need to be delivered further.
But an Item cannot be expected to have total scene awareness; so now,
the delivery is "done" only when all eventpoints in the original event
are accepted. This behavior has been working well enough already due to
logic in QQuickWindow that iterates the points and accepts them if the
event is accepted; but it seems appropriate to move this enforcement
into QPointerEvent itself. Making setAccepted() virtual gives us a
useful degree of freedom.
Event-handling code should alternatively use QEventPoint:setAccepted()
or QPointerEvent::setExclusiveGrabber() to take resonsibility for only
a subset of the touchpoints.
Another way to put it is that we treat QPointerEvent::setAccepted() as a
convenience method: accepting the QEventPoints is what counts (at least
in Qt Quick).
Change-Id: Icec42dc980f407bb5116f5c0852c051a4521105a
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
-rw-r--r-- | src/corelib/kernel/qcoreevent.cpp | 4 | ||||
-rw-r--r-- | src/corelib/kernel/qcoreevent.h | 2 | ||||
-rw-r--r-- | src/gui/kernel/qevent.cpp | 10 | ||||
-rw-r--r-- | src/gui/kernel/qevent.h | 1 | ||||
-rw-r--r-- | tests/auto/gui/kernel/qtouchevent/tst_qtouchevent.cpp | 10 |
5 files changed, 26 insertions, 1 deletions
diff --git a/src/corelib/kernel/qcoreevent.cpp b/src/corelib/kernel/qcoreevent.cpp index fbea252bc0..6fdc2a94af 100644 --- a/src/corelib/kernel/qcoreevent.cpp +++ b/src/corelib/kernel/qcoreevent.cpp @@ -375,6 +375,10 @@ QEvent::~QEvent() For convenience, the accept flag can also be set with accept(), and cleared with ignore(). + + \note Accepting a QPointerEvent implicitly + \l {QEventPoint::setAccepted()}{accepts} all the + \l {QPointerEvent::points()}{points} that the event carries. */ /*! diff --git a/src/corelib/kernel/qcoreevent.h b/src/corelib/kernel/qcoreevent.h index 0458719f40..721342a66d 100644 --- a/src/corelib/kernel/qcoreevent.h +++ b/src/corelib/kernel/qcoreevent.h @@ -300,7 +300,7 @@ public: inline Type type() const { return static_cast<Type>(t); } inline bool spontaneous() const { return spont; } - inline void setAccepted(bool accepted) { m_accept = accepted; } + inline virtual void setAccepted(bool accepted) { m_accept = accepted; } inline bool isAccepted() const { return m_accept; } inline void accept() { m_accept = true; } diff --git a/src/gui/kernel/qevent.cpp b/src/gui/kernel/qevent.cpp index 51100f56ff..5938942399 100644 --- a/src/gui/kernel/qevent.cpp +++ b/src/gui/kernel/qevent.cpp @@ -684,6 +684,16 @@ bool QPointerEvent::allPointsAccepted() const } /*! + \reimp +*/ +void QPointerEvent::setAccepted(bool accepted) +{ + QEvent::setAccepted(accepted); + for (auto &p : m_points) + p.setAccepted(accepted); +} + +/*! Returns the source device from which this event originates. This is the same as QInputEvent::device() but typecast for convenience. diff --git a/src/gui/kernel/qevent.h b/src/gui/kernel/qevent.h index 07d5be51c6..fe728bc40c 100644 --- a/src/gui/kernel/qevent.h +++ b/src/gui/kernel/qevent.h @@ -220,6 +220,7 @@ public: virtual bool isUpdateEvent() const { return false; } virtual bool isEndEvent() const { return false; } bool allPointsAccepted() const; + virtual void setAccepted(bool accepted) override; QObject *exclusiveGrabber(const QEventPoint &point) const; void setExclusiveGrabber(const QEventPoint &point, QObject *exclusiveGrabber); QList<QPointer <QObject>> passiveGrabbers(const QEventPoint &point) const; diff --git a/tests/auto/gui/kernel/qtouchevent/tst_qtouchevent.cpp b/tests/auto/gui/kernel/qtouchevent/tst_qtouchevent.cpp index 8d07718156..3f87c21f51 100644 --- a/tests/auto/gui/kernel/qtouchevent/tst_qtouchevent.cpp +++ b/tests/auto/gui/kernel/qtouchevent/tst_qtouchevent.cpp @@ -442,6 +442,7 @@ void tst_QTouchEvent::touchEventAcceptedByDefault() touchEvent.ignore(); QVERIFY(QApplication::sendEvent(&touchWidget, &touchEvent)); QVERIFY(touchEvent.isAccepted()); + QVERIFY(touchEvent.allPointsAccepted()); } // QGraphicsView @@ -470,6 +471,7 @@ void tst_QTouchEvent::touchEventAcceptedByDefault() (QList<QEventPoint>() << touchPoint)); QVERIFY(QApplication::sendEvent(view.viewport(), &touchEvent)); QVERIFY(touchEvent.isAccepted()); + QVERIFY(touchEvent.allPointsAccepted()); QVERIFY(item.seenTouchBegin); } } @@ -496,6 +498,7 @@ void tst_QTouchEvent::touchBeginPropagatesWhenIgnored() touchPoints); QVERIFY(QApplication::sendEvent(&grandchild, &touchEvent)); QVERIFY(touchEvent.isAccepted()); + QVERIFY(touchEvent.allPointsAccepted()); QVERIFY(grandchild.seenTouchBegin); QVERIFY(child.seenTouchBegin); QVERIFY(!window.seenTouchBegin); @@ -510,6 +513,7 @@ void tst_QTouchEvent::touchBeginPropagatesWhenIgnored() touchEvent.ignore(); QVERIFY(QApplication::sendEvent(&grandchild, &touchEvent)); QVERIFY(touchEvent.isAccepted()); + QVERIFY(touchEvent.allPointsAccepted()); QVERIFY(!grandchild.seenTouchBegin); QVERIFY(child.seenTouchBegin); QVERIFY(!window.seenTouchBegin); @@ -544,6 +548,7 @@ void tst_QTouchEvent::touchBeginPropagatesWhenIgnored() (QList<QEventPoint>() << touchPoint)); QVERIFY(QApplication::sendEvent(view.viewport(), &touchEvent)); QVERIFY(touchEvent.isAccepted()); + QVERIFY(touchEvent.allPointsAccepted()); QVERIFY(grandchild.seenTouchBegin); QVERIFY(child.seenTouchBegin); QVERIFY(!root.seenTouchBegin); @@ -578,6 +583,7 @@ void tst_QTouchEvent::touchBeginPropagatesWhenIgnored() (QList<QEventPoint>() << touchPoint)); QVERIFY(QApplication::sendEvent(view.viewport(), &touchEvent)); QVERIFY(touchEvent.isAccepted()); + QVERIFY(touchEvent.allPointsAccepted()); QVERIFY(!grandchild.seenTouchBegin); QVERIFY(child.seenTouchBegin); QVERIFY(!root.seenTouchBegin); @@ -604,6 +610,7 @@ void tst_QTouchEvent::touchUpdateAndEndNeverPropagate() touchPoints); QVERIFY(QApplication::sendEvent(&child, &touchBeginEvent)); QVERIFY(touchBeginEvent.isAccepted()); + QVERIFY(touchBeginEvent.allPointsAccepted()); QVERIFY(child.seenTouchBegin); QVERIFY(!window.seenTouchBegin); @@ -614,6 +621,7 @@ void tst_QTouchEvent::touchUpdateAndEndNeverPropagate() touchPoints); QVERIFY(QApplication::sendEvent(&child, &touchUpdateEvent)); QVERIFY(!touchUpdateEvent.isAccepted()); + QVERIFY(!touchBeginEvent.allPointsAccepted()); QVERIFY(child.seenTouchUpdate); QVERIFY(!window.seenTouchUpdate); @@ -624,6 +632,7 @@ void tst_QTouchEvent::touchUpdateAndEndNeverPropagate() touchPoints); QVERIFY(QApplication::sendEvent(&child, &touchEndEvent)); QVERIFY(!touchEndEvent.isAccepted()); + QVERIFY(!touchBeginEvent.allPointsAccepted()); QVERIFY(child.seenTouchEnd); QVERIFY(!window.seenTouchEnd); } @@ -657,6 +666,7 @@ void tst_QTouchEvent::touchUpdateAndEndNeverPropagate() (QList<QEventPoint>() << touchPoint)); QVERIFY(QApplication::sendEvent(view.viewport(), &touchBeginEvent)); QVERIFY(touchBeginEvent.isAccepted()); + QVERIFY(touchBeginEvent.allPointsAccepted()); QVERIFY(child.seenTouchBegin); QVERIFY(!root.seenTouchBegin); |