Don't allow modifiers for predefined shortcuts

MrKepzie · Jun 27, 2014 · 70c399e · 70c399e
1 parent e4e81c7
commit 70c399e
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 13 deletions.
diff --git a/Global/GitVersion.h b/Global/GitVersion.h
@@ -1,5 +1,5 @@
 #ifndef NATRON_GITVERSION_H_
 #define NATRON_GITVERSION_H_
 #define GIT_BRANCH "workshop"
-#define GIT_COMMIT "c8a1ccee143a3ed8a49969a1b466cd8655b53a1b"
+#define GIT_COMMIT "e4e81c75cc1ccec144cfa89d253661b8561c25a6"
 #endif
diff --git a/Gui/NodeGraph.cpp b/Gui/NodeGraph.cpp
@@ -1168,11 +1168,17 @@ void NodeGraph::keyPressEvent(QKeyEvent *e){
     if (e->key() == Qt::Key_Space) {
         QKeyEvent* ev = new QKeyEvent(QEvent::KeyPress,Qt::Key_Space,Qt::NoModifier);
         QCoreApplication::postEvent(parentWidget(),ev);
-    } else if (e->key() == Qt::Key_R) {
+    } else if (e->key() == Qt::Key_R && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         _gui->createReader();
-    } else if (e->key() == Qt::Key_W) {
+    } else if (e->key() == Qt::Key_W && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         _gui->createWriter();
-    } else if (e->key() == Qt::Key_Backspace || e->key() == Qt::Key_Delete) {
+    } else if ((e->key() == Qt::Key_Backspace || e->key() == Qt::Key_Delete) && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         /*delete current node.*/
         if (_selectedBackDrop) {
             _undoStack->setActive();
@@ -1181,23 +1187,41 @@ void NodeGraph::keyPressEvent(QKeyEvent *e){
         } else {
             deleteSelectedNode();
         }
-    } else if (e->key() == Qt::Key_P) {
+    } else if (e->key() == Qt::Key_P && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         forceRefreshAllPreviews();
-    } else if (e->key() == Qt::Key_C && e->modifiers().testFlag(Qt::ControlModifier)) {
+    } else if (e->key() == Qt::Key_C && e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         copySelectedNode();
-    } else if (e->key() == Qt::Key_V && e->modifiers().testFlag(Qt::ControlModifier)) {
+    } else if (e->key() == Qt::Key_V && e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         pasteNodeClipBoard();
-    } else if (e->key() == Qt::Key_X && e->modifiers().testFlag(Qt::ControlModifier)) {
+    } else if (e->key() == Qt::Key_X && e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         cutSelectedNode();
-    } else if (e->key() == Qt::Key_C && e->modifiers().testFlag(Qt::AltModifier)) {
+    } else if (e->key() == Qt::Key_C && e->modifiers().testFlag(Qt::AltModifier)
+               && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)) {
         duplicateSelectedNode();
-    } else if (e->key() == Qt::Key_K && e->modifiers().testFlag(Qt::AltModifier) && !e->modifiers().testFlag(Qt::ShiftModifier)) {
+    } else if (e->key() == Qt::Key_K && e->modifiers().testFlag(Qt::AltModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::ControlModifier)) {
         cloneSelectedNode();
-    } else if (e->key() == Qt::Key_K && e->modifiers().testFlag(Qt::AltModifier) && e->modifiers().testFlag(Qt::ShiftModifier)) {
+    } else if (e->key() == Qt::Key_K && e->modifiers().testFlag(Qt::AltModifier)
+               && e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::ControlModifier)) {
         decloneSelectedNode();
-    } else if (e->key() == Qt::Key_F) {
+    } else if (e->key() == Qt::Key_F && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         centerOnAllNodes();
-    } else if (e->key() == Qt::Key_H) {
+    } else if (e->key() == Qt::Key_H && !e->modifiers().testFlag(Qt::ControlModifier)
+               && !e->modifiers().testFlag(Qt::ShiftModifier)
+               && !e->modifiers().testFlag(Qt::AltModifier)) {
         toggleConnectionHints();
     }