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

Not working shortcut in Openbox #138

Closed
zotho opened this issue Feb 2, 2019 · 6 comments
Closed

Not working shortcut in Openbox #138

zotho opened this issue Feb 2, 2019 · 6 comments
Labels

Comments

@zotho
Copy link

zotho commented Feb 2, 2019

I can't close window using keyboard with Ctrl+Q.

I edit in:

ksnip/src/gui/MainWindow.cpp line 285

//mQuitAction->setShortcut(QKeySequence::Quit);
mQuitAction->setShortcut(QKeySequence(Qt::CTRL + Qt::Key_Q));

And works for me.

@DamirPorobic
Copy link
Member

That's interesting, on other distros this works. Might be an issue with Qt for Openbox, need to investigate that further. In other Qt based application the Ctrl+Q shortcut works?

@DamirPorobic
Copy link
Member

What system language are you using?

@zotho
Copy link
Author

zotho commented Feb 3, 2019

I used Russian. I changed system language to English (and reload pc) but problem remained. Ctrl+Q works in other Qt applications (Qt Creator, QCAD, qpdfview, qBittorrent, etc.). Also does not work Alt+F for open File menu.
ksnip_20190203-144936

@DamirPorobic
Copy link
Member

Only Alt+F doesn't work or other shortcuts too? Interesting thing, seems to affect only Openbox so far. Also interesting that the shortcut doesn't show up at all in the menu. Could be the Openbox Qt version doesn't map Ctrl+Q to Quit.

Do you build from source? Have you tried using the AppImage?

@zotho
Copy link
Author

zotho commented Feb 3, 2019

Yes, only Alt+F...
Alt+E, Alt+O and Alt+H work fine (they have underscore same letter, but "File" doesn't).
I build it from AUR (my system Manjaro/Openbox). When I build a program from git source I fixed QKeySequence::Quit to QKeySequence(Qt::CTRL + Qt::Key_Q) in the file and it work.

@DamirPorobic
Copy link
Member

Fixed with #308

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants