diff options
| author | Jan Arve Saether <jan-arve.saether@qt.io> | 2018-04-16 15:20:40 +0200 |
|---|---|---|
| committer | Jan Arve Sæther <jan-arve.saether@qt.io> | 2018-04-18 13:28:44 +0000 |
| commit | be4eaca053eba747c0458e37ce68ae3db1e6a288 (patch) | |
| tree | f477d792f7905db62b471efd580d63f9b0c194bc | |
| parent | c6ee1899eaa50dae56c07b6c4b3490cc5aeb60e6 (diff) | |
Do not ignore MenuItem on macOS
The idea was probably to ignore it since macOS already provides
accessibility for a menu item in its native system menu. However, a Qt
Quick Controls2 Menu will instead show a non-native menu, which should
not be ignored.
Task-number: QTBUG-63522
Change-Id: Ib5ae16ad991ebd7a18fa73b8f576f20b1c14d4c8
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@qt.io>
| -rw-r--r-- | src/plugins/platforms/cocoa/qcocoaaccessibility.mm | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/plugins/platforms/cocoa/qcocoaaccessibility.mm b/src/plugins/platforms/cocoa/qcocoaaccessibility.mm index 8b6dcb35a6..654e6210c8 100644 --- a/src/plugins/platforms/cocoa/qcocoaaccessibility.mm +++ b/src/plugins/platforms/cocoa/qcocoaaccessibility.mm @@ -226,7 +226,6 @@ bool shouldBeIgnored(QAccessibleInterface *interface) const QAccessible::Role role = interface->role(); if (role == QAccessible::Border || // QFrame role == QAccessible::Application || // We use the system-provided application element. - role == QAccessible::MenuItem || // The system also provides the menu items. role == QAccessible::ToolBar || // Access the tool buttons directly. role == QAccessible::Pane || // Scroll areas. role == QAccessible::Client) // The default for QWidget. |