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

Move tab focus specific functionality from QQuickControl to QQuickItem

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Fixed
    • Not Evaluated
    • 6.8
    • None
    • Quick: Other
    • 43625c714 (dev), 1ca992885 (dev), 64234dfc7 (dev), b37a53798 (dev), a7741ce0f (dev), 99ce38daa (dev), a85791ddf (dev), 58a40b1ab (dev), 57d7c2086 (6.7), eaaf383b5 (6.7), ac0bbf86c (6.7), 881ba6682 (6.7), 644ee4d23 (dev), 12c51fa90 (6.7), fc8799f24 (dev)

    Description

      This way, all QQuickItem subclasses can have the same focus behaviour and API. Less code to maintain and more consistent API

      Suggestion

      Move

      • QQuickControl::focusReason
      • QQuickControl::focusPolicy

      to QQuickItem

      • make sure that focusReason is properly set when key/mouse events arrive and demands a focus change

       

      Properties that become redundant after the above change (and should probably be deprecated):

      • QQuickItem::activeFocusOnTab
      • QQuickTextInput::activeFocusOnPress
      • QQuickTextEdit::activeFocusOnPress

      Should probably consolidate the code of those two properties into one (calling setActiveFocusOnTab() will set the TabFocus bit into the focusPolicy member, and the same for the other two)

       

      See also https://codereview.qt-project.org/c/qt/qtdeclarative/+/383968

      Attachments

        Issue Links

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

          Activity

            People

              dorisverria Doris Verria
              smd Jan Arve
              Votes:
              0 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: