Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No action shortcut is shown on non-menubar menus with Qt 5.10 #1426

Closed
tsujan opened this issue Dec 20, 2017 · 2 comments
Closed

No action shortcut is shown on non-menubar menus with Qt 5.10 #1426

tsujan opened this issue Dec 20, 2017 · 2 comments

Comments

@tsujan
Copy link
Member

tsujan commented Dec 20, 2017

As the title says: With Qt 5.10, action shortcuts aren't shown on menus anymore, unless they are menubar menus. For example, if the menubar of pcmanfm-qt is hidden, there are no action shortcuts on menus of the menu-button. Or if the view of Kate/FeatherPad/Qt Creator is right clicked, no shortcut is shown on the context menu.

Apparently, it makes no difference whether the Qt5 app is recompiled against Qt 5.10 or not.

And why did I report this here? Because it doesn't happen under KDE!! I have no idea about the cause of this magic.

@tsujan
Copy link
Member Author

tsujan commented Dec 20, 2017

Ah, found the son of a... This was the silliest change in Qt ever! Under LXQtPlatformTheme::themeHint(), this should be added for Qt-5.10:

    case QPlatformTheme::ShowShortcutsInContextMenus:
        return QVariant(true);

The ridiculous thing is that QPlatformTheme::ShowShortcutsInContextMenus should be only for context menus. Do Qt devs see all non-menubar menus as context menus now?!

EDIT: See http://doc.qt.io/qt-5/qstylehints.html#showShortcutsInContextMenus-prop.

tsujan added a commit to lxqt/lxqt-qtplugin that referenced this issue Dec 20, 2017
Fixes lxqt/lxqt#1426.

ShowShortcutsInContextMenus is false by default and, contrary to it name, it isn't just about context menus.
@tsujan
Copy link
Member Author

tsujan commented Dec 20, 2017

Off topic but related:

Quoting the commit message of https://codereview.qt-project.org/#/c/141647/:

"... Windows and GNOME desktop environments almost never use them. Only KDE appears to do so commonly; default accordingly."

IMO, in the long run, it won't be good if Qt devs think that it's enough to consider only Gnome and KDE on Linux.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Issues
  
Translations and L10N
Development

No branches or pull requests

1 participant