MidiEditor icon indicating copy to clipboard operation
MidiEditor copied to clipboard

Please allow rebinding the keyboard shortcuts to something else

Open io7m opened this issue 7 years ago • 2 comments

The draw note tool and the select note tool... The two tools that are almost certainly used more frequently than any other tools, and that need to be used in constant alternation (because it's not possible to select an existing note with the draw tool enabled), have hardcoded keyboard shortcuts at opposite ends of the keyboard!

Repetitive. Strain. Injury.

I don't care if there's even a UI for it, setting them via a config file would be fine!

I'll be redoing these shortcuts to something somewhat less wrist-destroying in my own fork (with no attempt made to make it configurable).

io7m avatar Apr 10 '18 19:04 io7m

diff --git a/src/gui/MainWindow.cpp b/src/gui/MainWindow.cpp
index efa841d..928b502 100644
--- a/src/gui/MainWindow.cpp
+++ b/src/gui/MainWindow.cpp
@@ -2172,7 +2172,7 @@ QWidget *MainWindow::setupActions(QWidget *parent){
 
        StandardTool *tool = new StandardTool();
        Tool::setCurrentTool(tool);
-       stdToolAction = new ToolButton(tool, QKeySequence(Qt::Key_F1), toolsToolsMenu);
+       stdToolAction = new ToolButton(tool, QKeySequence(Qt::Key_Z), toolsToolsMenu);
        toolsToolsMenu->addAction(stdToolAction);
        tool->buttonClick();
 
@@ -2201,7 +2201,7 @@ QWidget *MainWindow::setupActions(QWidget *parent){
 
        toolsToolsMenu->addSeparator();
 
-       QAction *newNoteAction = new ToolButton(new NewNoteTool(), QKeySequence(Qt::Key_F10), toolsToolsMenu);
+       QAction *newNoteAction = new ToolButton(new NewNoteTool(), QKeySequence(Qt::Key_X), toolsToolsMenu);
        toolsToolsMenu->addAction(newNoteAction);
        QAction *removeNotesAction = new ToolButton(new EraserTool(), QKeySequence(Qt::Key_F11), toolsToolsMenu);
        toolsToolsMenu->addAction(removeNotesAction);

io7m avatar Apr 10 '18 21:04 io7m

In case I haven't hammered home the point enough: This ticket is intended in good humour. :grin:

I do think configurable shortcuts are needed though.

io7m avatar Apr 10 '18 21:04 io7m