QQuickMenu: fix key navigation

Skip non-focusable separators, and use a key focus reason
(Qt::TabFocusReason & Qt::BacktabFocusReason) to give the
items visual focus.

[ChangeLog][Controls][Menu] Fixed key navigation to skip separators.

Change-Id: I99affabc50703c7363ab8146e5ced9b45111de00
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>

1 files changed, 74 insertions, 3 deletions

diff --git a/tests/auto/menu/tst_menu.cpp b/tests/auto/menu/tst_menu.cpp
index 1e6eaceb..19a4006b 100644
--- a/tests/auto/menu/tst_menu.cpp
+++ b/tests/auto/menu/tst_menu.cpp
@@ -165,7 +165,8 @@ void tst_menu::contextMenuKeyboard()
     QQuickMenu *menu = window->property("menu").value<QQuickMenu*>();
     QCOMPARE(menu->contentItem()->property("currentIndex"), QVariant(-1));
 
-    QQuickItem *firstItem = menu->itemAt(0);
+    QQuickMenuItem *firstItem = qobject_cast<QQuickMenuItem *>(menu->itemAt(0));
+    QVERIFY(firstItem);
     QSignalSpy visibleSpy(menu, SIGNAL(visibleChanged()));
 
     menu->setFocus(true);
@@ -178,12 +179,18 @@ void tst_menu::contextMenuKeyboard()
 
     QTest::keyClick(window, Qt::Key_Tab);
     QVERIFY(firstItem->hasActiveFocus());
+    QVERIFY(firstItem->hasVisualFocus());
+    QCOMPARE(firstItem->focusReason(), Qt::TabFocusReason);
     QCOMPARE(menu->contentItem()->property("currentIndex"), QVariant(0));
 
-    QQuickItem *secondItem = menu->itemAt(1);
+    QQuickMenuItem *secondItem = qobject_cast<QQuickMenuItem *>(menu->itemAt(1));
+    QVERIFY(secondItem);
     QTest::keyClick(window, Qt::Key_Tab);
     QVERIFY(!firstItem->hasActiveFocus());
+    QVERIFY(!firstItem->hasVisualFocus());
     QVERIFY(secondItem->hasActiveFocus());
+    QVERIFY(secondItem->hasVisualFocus());
+    QCOMPARE(secondItem->focusReason(), Qt::TabFocusReason);
     QCOMPARE(menu->contentItem()->property("currentIndex"), QVariant(1));
 
     QSignalSpy secondTriggeredSpy(secondItem, SIGNAL(triggered()));
@@ -193,7 +200,9 @@ void tst_menu::contextMenuKeyboard()
     QVERIFY(!menu->isVisible());
     QVERIFY(!window->overlay()->childItems().contains(menu->contentItem()));
     QVERIFY(!firstItem->hasActiveFocus());
+    QVERIFY(!firstItem->hasVisualFocus());
     QVERIFY(!secondItem->hasActiveFocus());
+    QVERIFY(!secondItem->hasVisualFocus());
     QCOMPARE(menu->contentItem()->property("currentIndex"), QVariant(-1));
 
     menu->open();
@@ -201,26 +210,59 @@ void tst_menu::contextMenuKeyboard()
     QVERIFY(menu->isVisible());
     QVERIFY(window->overlay()->childItems().contains(menu->contentItem()->parentItem()));
     QVERIFY(!firstItem->hasActiveFocus());
+    QVERIFY(!firstItem->hasVisualFocus());
     QVERIFY(!secondItem->hasActiveFocus());
+    QVERIFY(!secondItem->hasVisualFocus());
     QCOMPARE(menu->contentItem()->property("currentIndex"), QVariant(-1));
 
     QTest::keyClick(window, Qt::Key_Down);
     QVERIFY(firstItem->hasActiveFocus());
+    QVERIFY(firstItem->hasVisualFocus());
+    QCOMPARE(firstItem->focusReason(), Qt::TabFocusReason);
 
     QTest::keyClick(window, Qt::Key_Down);
     QVERIFY(secondItem->hasActiveFocus());
+    QVERIFY(secondItem->hasVisualFocus());
+    QCOMPARE(secondItem->focusReason(), Qt::TabFocusReason);
 
     QTest::keyClick(window, Qt::Key_Down);
-    QQuickItem *thirdItem = menu->itemAt(2);
+    QQuickMenuItem *thirdItem = qobject_cast<QQuickMenuItem *>(menu->itemAt(2));
+    QVERIFY(thirdItem);
     QVERIFY(!firstItem->hasActiveFocus());
+    QVERIFY(!firstItem->hasVisualFocus());
     QVERIFY(!secondItem->hasActiveFocus());
+    QVERIFY(!secondItem->hasVisualFocus());
     QVERIFY(thirdItem->hasActiveFocus());
+    QVERIFY(thirdItem->hasVisualFocus());
+    QCOMPARE(thirdItem->focusReason(), Qt::TabFocusReason);
 
     // Key navigation shouldn't wrap by default.
     QTest::keyClick(window, Qt::Key_Down);
     QVERIFY(!firstItem->hasActiveFocus());
+    QVERIFY(!firstItem->hasVisualFocus());
     QVERIFY(!secondItem->hasActiveFocus());
+    QVERIFY(!secondItem->hasVisualFocus());
     QVERIFY(thirdItem->hasActiveFocus());
+    QVERIFY(thirdItem->hasVisualFocus());
+    QCOMPARE(thirdItem->focusReason(), Qt::TabFocusReason);
+
+    QTest::keyClick(window, Qt::Key_Up);
+    QVERIFY(!firstItem->hasActiveFocus());
+    QVERIFY(!firstItem->hasVisualFocus());
+    QVERIFY(secondItem->hasActiveFocus());
+    QVERIFY(secondItem->hasVisualFocus());
+    QCOMPARE(secondItem->focusReason(), Qt::BacktabFocusReason);
+    QVERIFY(!thirdItem->hasActiveFocus());
+    QVERIFY(!thirdItem->hasVisualFocus());
+
+    QTest::keyClick(window, Qt::Key_Backtab);
+    QVERIFY(firstItem->hasActiveFocus());
+    QVERIFY(firstItem->hasVisualFocus());
+    QCOMPARE(firstItem->focusReason(), Qt::BacktabFocusReason);
+    QVERIFY(!secondItem->hasActiveFocus());
+    QVERIFY(!secondItem->hasVisualFocus());
+    QVERIFY(!thirdItem->hasActiveFocus());
+    QVERIFY(!thirdItem->hasVisualFocus());
 
     QTest::keyClick(window, Qt::Key_Escape);
     QCOMPARE(visibleSpy.count(), 4);
@@ -317,6 +359,35 @@ void tst_menu::menuSeparator()
     QTest::mouseClick(window, Qt::LeftButton, Qt::NoModifier,
         saveMenuItem->mapToScene(QPointF(saveMenuItem->width() / 2, saveMenuItem->height() / 2)).toPoint());
     QTRY_VERIFY(!menu->isVisible());
+
+    menu->open();
+    QVERIFY(menu->isVisible());
+
+    // Key navigation skips separators
+    QTest::keyClick(window, Qt::Key_Down);
+    QVERIFY(newMenuItem->hasActiveFocus());
+    QVERIFY(newMenuItem->hasVisualFocus());
+    QCOMPARE(newMenuItem->focusReason(), Qt::TabFocusReason);
+
+    QTest::keyClick(window, Qt::Key_Down);
+    QVERIFY(saveMenuItem->hasActiveFocus());
+    QVERIFY(saveMenuItem->hasVisualFocus());
+    QCOMPARE(saveMenuItem->focusReason(), Qt::TabFocusReason);
+
+    QTest::keyClick(window, Qt::Key_Down);
+    QVERIFY(saveMenuItem->hasActiveFocus());
+    QVERIFY(saveMenuItem->hasVisualFocus());
+    QCOMPARE(saveMenuItem->focusReason(), Qt::TabFocusReason);
+
+    QTest::keyClick(window, Qt::Key_Up);
+    QVERIFY(newMenuItem->hasActiveFocus());
+    QVERIFY(newMenuItem->hasVisualFocus());
+    QCOMPARE(newMenuItem->focusReason(), Qt::BacktabFocusReason);
+
+    QTest::keyClick(window, Qt::Key_Up);
+    QVERIFY(newMenuItem->hasActiveFocus());
+    QVERIFY(newMenuItem->hasVisualFocus());
+    QCOMPARE(newMenuItem->focusReason(), Qt::BacktabFocusReason);
 }
 
 void tst_menu::repeater()