diff options
author | Shawn Rutledge <shawn.rutledge@qt.io> | 2017-05-02 11:34:19 +0200 |
---|---|---|
committer | Shawn Rutledge <shawn.rutledge@qt.io> | 2017-05-10 06:40:39 +0000 |
commit | ab91e7fa02a562d80fd0747f28a60e00c3b45a01 (patch) | |
tree | f0cf333802921b3706bbf1b2e892a906e336112e /src/quick/items/qquickwindow.cpp | |
parent | d84af8b81543e200db3041f27eadc2df5c62055e (diff) |
Default QQuickItem::acceptTouchEvents to true until Qt 6
This is a partial revert of 1457df74f4c1d770e1e820de8cd082be1bd2489e
to avoid making a mandatory API change so soon.
Change-Id: I05040579fa36d3dc5ef7616861f6d17adf500d2c
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
Diffstat (limited to 'src/quick/items/qquickwindow.cpp')
-rw-r--r-- | src/quick/items/qquickwindow.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/quick/items/qquickwindow.cpp b/src/quick/items/qquickwindow.cpp index 289e1eabec..9c889d54f5 100644 --- a/src/quick/items/qquickwindow.cpp +++ b/src/quick/items/qquickwindow.cpp @@ -2668,9 +2668,15 @@ void QQuickWindowPrivate::updateFilteringParentItems(const QVector<QQuickItem *> } filteringParentItems.clear(); for (QQuickItem *item : targetItems) { +#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0) + bool acceptsTouchEvents = item->acceptTouchEvents(); +#else + // In versions prior to Qt 6, we can't trust item->acceptTouchEvents() here, because it defaults to true. + bool acceptsTouchEvents = false; +#endif QQuickItemPrivate *itemPriv = QQuickItemPrivate::get(item); - // If the item neither handles events nor has handlers which do, then it will never be a receiver, so filtering is irrelevant - if (!item->acceptedMouseButtons() && !item->acceptTouchEvents() && + // If the item neither handles events nor has handlers which do, then it will never be a receiver, so filtering is irrelevant. + if (!item->acceptedMouseButtons() && !acceptsTouchEvents && !(itemPriv->extra.isAllocated() && !itemPriv->extra->pointerHandlers.isEmpty())) continue; QQuickItem *parent = item->parentItem(); |