Skip to content
This repository has been archived by the owner on Jun 8, 2018. It is now read-only.

Change the shortcut to exit to Ctrl+Q. #301

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Change the shortcut to exit to Ctrl+Q. #301

wants to merge 1 commit into from

Conversation

orbea
Copy link
Contributor

@orbea orbea commented Sep 5, 2017

I find it more convenient to be able to exit with the keyboard than mouse so I suggest using Ctrl+Q since its a fairly standard keybind for closing GUI programs. Suggestions are of course welcome.

@orbea
Copy link
Contributor Author

orbea commented Sep 5, 2017

Looks like the buildbot is currently broken for unrelated issues? I do not run debian so I'm not really sure what to do about it...

@christopho
Copy link
Collaborator

Yes the build error is unrelated.

But I want to keep the QKeySequence::Quit shortcut also possible since it is likely to be the standard one on a lot of systems. Is there is a way to add the Ctrl+Q shortcut while keeping the QKeySequence::Quit also working?

@oclero
Copy link
Member

oclero commented Sep 6, 2017

you can set multiple QKeySequence on one QAction, just give it a list

@orbea
Copy link
Contributor Author

orbea commented Sep 6, 2017

I agree with the advice, I will see if I can do it better, but my programming experience is rather limited so I'll have to see if I can manage...

@orbea orbea closed this Nov 27, 2017
@christopho
Copy link
Collaborator

Reopening, we can do the change ourselves.

@christopho christopho reopened this Dec 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants