(Shift)Tab must not leave dialog/popover/menu

[Cris Dywan]

It must not be possible to leave a dialog/popover/menu using (Shift)Tab. The only expected ways to leave are Escape and any component or key in the popover which would result in closing it.

http://code.qt.io/cgit/qt/qtdeclarative.git/diff/?id=6328dd2d2743ed540efea89742261f24552d7611
5.6 alpha introduces private API, isTabFence, to confine focus to a container. We could introduce API on StyledItem to expose that and use it in popovers.

[Timo Jyrinki (timo-jyrinki)]

Testable from https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-060/

[Timo Jyrinki (timo-jyrinki)]

Ticket https://requests.ci-train.ubuntu.com/#/ticket/791

[Timo Jyrinki (timo-jyrinki)]

Scheduled for OTA-10 because it's only needed there together with UITK changes.

[Timo Jyrinki (timo-jyrinki)]

A related bug https://bugreports.qt.io/browse/QTBUG-50516

[Timo Jyrinki (timo-jyrinki)]

There's a second infinite loop fix WIP at https://codereview.qt-project.org/#/c/146886/

[Timo Jyrinki (timo-jyrinki)]

060 updated with patch set 2 from upstream, armhf build will be ready in 2h.

[Timo Jyrinki (timo-jyrinki)]

Moved to silo 23 for final builds, please retest!

[Timo Jyrinki (timo-jyrinki)]

It was successfully tested three days ago and QA was asked to add it into their queue despite britney warnings caused by -proposed but not this landing.

[Łukasz Zemczak (sil2100)]

This bug was fixed in the package qtdeclarative-opensource-src 5.4.1-1ubuntu11~overlay3 in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay

---------------

qtdeclarative-opensource-src (5.4.1-1ubuntu11~overlay3) vivid; urgency=medium

  * debian/patches/Add-isTabFence-private-flag.patch:
    - Tab should not leave dialog/popover/menu (LP: #1526208)
  * debian/patches/QQuickItem-fix-an-infinite-loop-nextItemInFocusChain.patch
    debian/patches/QQuickItem-fix-another-infinite-loop-in-nextItemInFo.patch:
    - Refine the previous patch (LP: #1526208)

 -- Timo Jyrinki <email address hidden> Sat, 30 Jan 2016 14:00:48 +0000

[Launchpad Janitor (janitor)]

This bug was fixed in the package qtdeclarative-opensource-src - 5.5.1-2ubuntu4

---------------
qtdeclarative-opensource-src (5.5.1-2ubuntu4) xenial; urgency=medium

  * debian/patches/Add-isTabFence-private-flag.patch:
    - Tab should not leave dialog/popover/menu (LP: #1526208)
  * debian/patches/QQuickItem-fix-an-infinite-loop-nextItemInFocusChain.patch
    debian/patches/QQuickItem-fix-another-infinite-loop-in-nextItemInFo.patch:
    - Refine the previous patch (LP: #1526208)

 -- Timo Jyrinki <email address hidden> Sat, 30 Jan 2016 14:06:58 +0000