Improved error message when trying to use a modifier twice in a shortcut

adobe · Oct 16, 2012 · 9d204d7 · 9d204d7
1 parent 0beaa9f
commit 9d204d7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/command/KeyBindingManager.js b/src/command/KeyBindingManager.js
@@ -126,13 +126,13 @@ define(function (require, exports, module) {
             right = right.trim().toLowerCase();
             var matched = (left.length > 0 && left === right);
             if (matched && previouslyFound) {
-                console.log("KeyBindingManager normalizeKeyDescriptorString() - Modifier defined twice: " + origDescriptor);
+                console.log("KeyBindingManager normalizeKeyDescriptorString() - Modifier " + left + " defined twice: " + origDescriptor);
             }
             return matched;
         }
 
         origDescriptor.split("-").forEach(function parseDescriptor(ele, i, arr) {
-            if (_compareModifierString("ctrl", ele, hasCtrl)) {
+            if (_compareModifierString("ctrl", ele, hasCtrl, origDescriptor)) {
                 if (brackets.platform === "mac") {
                     hasMacCtrl = true;
                 } else {