Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@YoshiMan
Copy link

enabled shortcut
extra treatment for mc os

enabled shortcut
extra treatment for mc os
@YoshiMan
Copy link
Author

please test under Mac OS

@annejan
Copy link
Member

annejan commented Dec 14, 2016

Travis-CI will fail on macOS when it finally gets to that @YoshiMan ;)

http://doc.qt.io/qt-5/qkeysequence.html#qt_set_sequence_auto_mnemonic
Note: This function is not declared in any of Qt's header files.
To use it in your application, declare the function prototype before calling it.

But it is available via on 4.8 . .

To test I added to mainwindow.h (before the namespace and class stuff)

#ifdef __APPLE__
#if QT_VERSION >= QT_VERSION_CHECK(5, 0, 0)
// http://doc.qt.io/qt-5/qkeysequence.html#qt_set_sequence_auto_mnemonic
void qt_set_sequence_auto_mnemonic(bool b);
#endif
#endif

Unfortunately this change does nothing for me since Cmd+Q already closes the app on macOS 😁

But I also tested on Windows VM and seems to work fine . .

@YoshiMan
Copy link
Author

Yeah. It would be better, if you could fix this. Thank you.

@annejan annejan merged commit 6cab167 into IJHack:master Dec 14, 2016
@YoshiMan YoshiMan deleted the feature/#258-CTRL-CMD-Q-closes-the- branch December 14, 2016 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants