Changeset 8952 in josm for trunk/src

Timestamp:

2015-10-27T14:03:41+01:00 (9 years ago)

Author:

Don-vip

Message:

see #12012 - add log message for shortcut reassignments

File:

• trunk/src/org/openstreetmap/josm/tools/Shortcut.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/tools/Shortcut.java

@@ -434 +434 @@
     private static Shortcut registerShortcut(String shortText, String longText, int requestedKey, int requestedGroup, Integer modifier) {
         doInit();
-        Integer defaultModifier = findModifier(requestedGroup, modifier);
         if (shortcuts.containsKey(shortText)) { // a re-register? maybe a sc already read from the preferences?
             Shortcut sc = shortcuts.get(shortText);
@@ -441 +440 @@
             return sc;
         }
+        Integer defaultModifier = findModifier(requestedGroup, modifier);
         Shortcut conflict = findShortcut(requestedKey, defaultModifier);
         if (conflict != null) {
@@ -447 +447 @@
                 int newmodifier = findNewOsxModifier(requestedGroup);
                 if (findShortcut(requestedKey, newmodifier) == null) {
+                    Main.info("Reassigning OSX shortcut '" + shortText + "' from Meta to Ctrl because of conflict with " + conflict);
                     return reassignShortcut(shortText, longText, requestedKey, conflict, requestedGroup, requestedKey, newmodifier);
                 }
@@ -454 +455 @@
                     int newmodifier = getGroupModifier(m);
                     if (findShortcut(k, newmodifier) == null) {
+                        Main.info("Reassigning shortcut '" + shortText + "' from " + modifier + " to " + newmodifier +
+                                " because of conflict with " + conflict);
                         return reassignShortcut(shortText, longText, requestedKey, conflict, m, k, newmodifier);
                     }