Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-113404

Wayland: Gestures are not delivered to correct top level widget

    XMLWordPrintable

Details

    • Linux/Wayland
    • f7a2d5233 (dev), ae380d343 (dev), 10c3dd87d (dev), 98d1534df (6.7), 1353b9cf3 (6.7), e0ee9cfe3 (6.7), 12cc65053 (tqtc/lts-6.5), 87b6157b9 (tqtc/lts-6.5), 74f06bc0d (dev), 04b8b61ce (dev), 8e3344778 (6.8), 0dfa96aac (6.7), 84673a284 (tqtc/lts-6.5)

    Description

      When a client application with two or more top level widgets is run in a Wayland compositor the gesture events are not delivered to the correct top level widget.

      QGestureManager::deliverEvents() uses the gesture's hotspot to find the top-level widget. It looks like the Wayland windows contain the same coordinates, so it is picking what it thinks is the top-level widget. If the QGestureManager::deliverEvents() is forced to use the gesture's context from m_gestureOwners, instead of the hotspot, the gesture is sent to the correct window.

      To reproduce, please run the attached example which consists of the compositor and the client. Use mouse and touch screen to click the both top level widgets.

      The QApplication::topLevelAt() with the global mouse event position for mousePressEvent() logging always shows "topLevelAtWidget=Right" even when clicking on the "Left" widget. When tapped with a touch screen the tap gesture is always showing "objectName=Right".

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-121015 tst_QGuiApplication::topLevelAt() failed on Wayland

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          resulted in

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-125878 Cannot pinch-to-zoom after the qt 6.7.1 upgrade [regression]

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-126313 QScroller gesture blocks mouse clicks

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-126703 QApplication::widgetAt() always returns nullptr

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed
          For Gerrit Dashboard: QTBUG-113404
          # Subject Branch Project Status CR V
          556775,1 client: Synthesize enter/leave event for popup in xdg-shell - 2 tqtc/lts-6.2 qt/tqtc-qtwayland Status: NEW +2 0
          554332,6 QGestureManager: use gesture owner target if topLevelAt returns nullptr dev qt/qtbase Status: MERGED +2 0
          554610,5 client: implement QWaylandScreen::topLevelAt() dev qt/qtwayland Status: MERGED +2 0
          556534,2 client: Synthesize enter/leave event for popup in xdg-shell - 2 dev qt/qtwayland Status: MERGED +2 0
          556631,2 client: Synthesize enter/leave event for popup in xdg-shell - 2 6.7 qt/qtwayland Status: MERGED +2 0
          556632,2 client: implement QWaylandScreen::topLevelAt() 6.7 qt/qtwayland Status: MERGED +2 0
          556698,2 QGestureManager: use gesture owner target if topLevelAt returns nullptr 6.7 qt/qtbase Status: MERGED +2 0
          556730,1 QGestureManager: use gesture owner target if topLevelAt returns nullptr tqtc/lts-6.5 qt/tqtc-qtbase Status: ABANDONED +2 0
          556732,2 client: Synthesize enter/leave event for popup in xdg-shell - 2 tqtc/lts-6.5 qt/tqtc-qtwayland Status: MERGED +2 0
          556733,2 client: implement QWaylandScreen::topLevelAt() tqtc/lts-6.5 qt/tqtc-qtwayland Status: MERGED +2 0
          556776,1 client: implement QWaylandScreen::topLevelAt() tqtc/lts-6.2 qt/tqtc-qtwayland Status: ABANDONED +2 0
          569969,4 widgets: Add QWidgetPrivate::widgetAt() dev qt/qtbase Status: ABANDONED 0 0
          569970,5 widget: correct receiver in QWidgetWindow::handleGestureEvent() dev qt/qtbase Status: ABANDONED 0 0
          571499,3 WIP: widget: pass the receiver widget to PressDelayHandler::pressed dev qt/qtbase Status: ABANDONED -2 0
          572059,4 Revert "QGestureManager: use gesture owner target if topLevelAt returns nullptr" dev qt/qtbase Status: MERGED +2 0
          572060,3 Revert "client: implement QWaylandScreen::topLevelAt()" dev qt/qtwayland Status: MERGED +2 0
          573458,2 Revert "client: implement QWaylandScreen::topLevelAt()" 6.8 qt/qtwayland Status: MERGED +2 0
          573566,2 Revert "QGestureManager: use gesture owner target if topLevelAt returns nullptr" 6.8 qt/qtbase Status: MERGED +2 0
          573614,2 Revert "client: implement QWaylandScreen::topLevelAt()" 6.7 qt/qtwayland Status: MERGED +2 0
          573698,2 Revert "client: implement QWaylandScreen::topLevelAt()" tqtc/lts-6.5 qt/tqtc-qtwayland Status: MERGED +2 0
          573956,2 Revert "QGestureManager: use gesture owner target if topLevelAt returns nullptr" 6.7 qt/qtbase Status: MERGED +2 0

          Activity

            People

              liaqi Liang Qi
              tuomas.vaarala Tuomas Vaarala
              Votes:
              1 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There is 1 open Gerrit change