GNOME Bugzilla – Bug 733816
Ctrl+n does not work
Last modified: 2016-03-31 13:22:07 UTC
Ctrl+n should click the new button. I'm making this dependent on the code refactorings to avoid unneeded merges.
The accelerator is available through Alt so Alt+N achieves that.
yes thats the mnemonic. But usually every application supports Ctrl+N for creating new things, users are therefore accustomed to it and use it intuitively. (At least I do...)
Created attachment 283567 [details] [review] app-window: Add Ctrl+N shortcut
Created attachment 283568 [details] [review] collection-toolbar: Move public functions up
Review of attachment 283567 [details] [review]: Some indication in the log about what ctrl+n does would be nice. :)
Review of attachment 283567 [details] [review]: isnt that a 'common abbreviation'? ;) I'll reupload in some minutes.
Review of attachment 283568 [details] [review]: if you move this patch up (before the other one), it will be smaller change as you can merge the click_new_button change into the other patch. :)
Review of attachment 283567 [details] [review]: "isnt that a 'common abbreviation'? ;)" abbreviation -> shortcut and even if its common, doesn't mean everyone knows about it. At least I didn't know its supported by everything. :)
Created attachment 283609 [details] [review] collection-toolbar: Move public functions up
Created attachment 283610 [details] [review] app-window: Add Ctrl+N shortcut Ctrl+N is a common shortcut for creating new things. Lets support it for creating a new VM.
Attachment 283609 [details] pushed as cdef60b - collection-toolbar: Move public functions up Attachment 283610 [details] pushed as 433c8af - app-window: Add Ctrl+N shortcut