search

FrancescoCeruti / linux-show-player

Linux Show Player - Cue player designed for stage productions
https://linux-show-player.org
GNU General Public License v3.0
205 stars 49 forks source link

Add ctrl-q as shortcut to quit application #194

Closed frederic-peters closed 4 years ago

frederic-peters commented 4 years ago

Hi, I am used to Ctrl-Q for quitting an application and that shortcut was not implemented, so here is the obvious/trivial patch. Thanks,

FrancescoCeruti commented 4 years ago

Hey, looks fine to me, but you should work with the development branch. I'm not going to merge new features in the current master, until the development branch is merged, and it's a lot of commits ahead.

Also you should use QKeySequence.Quit instead of a fixed value.

frederic-peters commented 4 years ago

Thanks for the review; I missed the develop branch; I rebased mine and changed it to use QKeySequence.Quit.