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

QScrollBar::createStandardContextMenu does not set QMenu parent vs. QLineEdit, QPlainTextEdit, QTextEdit

    XMLWordPrintable

Details

    • 8649e06a1 (dev), cf5276fbf (6.10)

    Description

      For consistent experience it might be good to have behaviour aligned between all the implementation of the util method createStandardContextMenu. So one can rely on patterns, instead having to learn for each widget what a method of the same name actually does.

      While the docs state otherwise, QLineEdit, QPlainTextEdit, QTextEdit, but also QWebEngineView (though detail undocumented) all set the view instance as parent (comp. also https://bugreports.qt.io/browse/QTBUG-121698).

      No clue about the transient window stuff. But would there not be need to have some info about the transient parent window for such popups? Like when there are multiple top-level windows with an app? In KDE people mentioned with Wayland with QMenu one should see to set parents, see https://community.kde.org/Guidelines_and_HOWTOs/Wayland_Porting_Notes#Popup_Menus

      There has been related discussion on the respeciive review https://codereview.qt-project.org/c/qt/qtbase/+/499622 but it seems unclear if the problem was fully investigated, as "Just tested" does not tell what and how?

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            qt.team.quick.subscriptions Qt Quick and Widgets Team
            kossebau Friedrich W. H. Kossebau
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes