Changeset 4958 in josm

Timestamp:

2012-02-16T19:12:00+01:00 (12 years ago)

Author:

stoecker

Message:

fix remaining shortcut deprecations

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/CopyCoordinatesAction.java (modified) (1 diff)
• actions/CreateCircleAction.java (modified) (1 diff)
• actions/CreateMultipolygonAction.java (modified) (1 diff)
• actions/JoinAreasAction.java (modified) (1 diff)
• actions/PurgeAction.java (modified) (1 diff)
• actions/SaveAsAction.java (modified) (1 diff)
• actions/UnselectAllAction.java (modified) (1 diff)
• actions/ViewportFollowToggleAction.java (modified) (1 diff)
• gui/dialogs/CommandStackDialog.java (modified) (1 diff)
• gui/dialogs/HistoryDialog.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/CopyCoordinatesAction.java

@@ -17 +17 @@
         super(tr("Copy Coordinates"), null,
                 tr("Copy coordinates of selected nodes to clipboard."),
-                Shortcut.registerShortcut("copy:coordinates", tr("Edit: {0}", tr("Copy Coordinates")), KeyEvent.VK_C, Shortcut.GROUP_MENU, Shortcut.SHIFT_DEFAULT),
+                Shortcut.registerShortcut("copy:coordinates", tr("Edit: {0}", tr("Copy Coordinates")),
+                KeyEvent.VK_C, Shortcut.GROUP_MENU+Shortcut.GROUPS_ALT1),
                 false);
         putValue("toolbar", "copy/coordinates");

trunk/src/org/openstreetmap/josm/actions/CreateCircleAction.java

@@ -43 +43 @@
     public CreateCircleAction() {
         super(tr("Create Circle"), "createcircle", tr("Create a circle from three selected nodes."),
-                Shortcut.registerShortcut("tools:createcircle", tr("Tool: {0}", tr("Create Circle")), KeyEvent.VK_O, Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true);
+            Shortcut.registerShortcut("tools:createcircle", tr("Tool: {0}", tr("Create Circle")),
+            KeyEvent.VK_O, Shortcut.GROUP_EDIT+Shortcut.GROUPS_ALT1), true);
         putValue("help", ht("/Action/CreateCircle"));
     }

trunk/src/org/openstreetmap/josm/actions/CreateMultipolygonAction.java

@@ -48 +48 @@
     public CreateMultipolygonAction() {
         super(tr("Create multipolygon"), "multipoly_create", tr("Create multipolygon."),
-                Shortcut.registerShortcut("tools:multipoly", tr("Tool: {0}", tr("Create multipolygon")),
-                        KeyEvent.VK_A, Shortcut.GROUP_DIRECT, KeyEvent.ALT_DOWN_MASK | KeyEvent.CTRL_DOWN_MASK ), true);
+            Shortcut.registerShortcut("tools:multipoly", tr("Tool: {0}", tr("Create multipolygon")),
+            KeyEvent.VK_A, Shortcut.GROUP_MENU+Shortcut.GROUPS_ALT2), true);
     }
     /**

trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

@@ -306 +306 @@
     // Adds the menu entry, Shortcuts, etc.
     public JoinAreasAction() {
-        super(tr("Join overlapping Areas"), "joinareas", tr("Joins areas that overlap each other"), Shortcut.registerShortcut("tools:joinareas", tr("Tool: {0}", tr("Join overlapping Areas")),
-                KeyEvent.VK_J, Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true);
+        super(tr("Join overlapping Areas"), "joinareas", tr("Joins areas that overlap each other"),
+        Shortcut.registerShortcut("tools:joinareas", tr("Tool: {0}", tr("Join overlapping Areas")),
+            KeyEvent.VK_J, Shortcut.GROUP_EDIT+Shortcut.GROUPS_ALT1), true);
     }
 

trunk/src/org/openstreetmap/josm/actions/PurgeAction.java

@@ -60 +60 @@
         /* translator note: other expressions for "purge" might be "forget", "clean", "obliterate", "prune" */
         super(tr("Purge..."), "purge",  tr("Forget objects but do not delete them on server when uploading."),
-                Shortcut.registerShortcut("system:purge", tr("Edit: {0}", tr("Purge")), KeyEvent.VK_P, Shortcut.GROUP_MENU, Shortcut.SHIFT_DEFAULT)
-                , true);
+                Shortcut.registerShortcut("system:purge", tr("Edit: {0}", tr("Purge")),
+                KeyEvent.VK_P, Shortcut.GROUP_MENU+Shortcut.GROUPS_ALT1),
+                true);
         putValue("help", HelpUtil.ht("/Action/Purge"));
     }

trunk/src/org/openstreetmap/josm/actions/SaveAsAction.java

@@ -24 +24 @@
     public SaveAsAction() {
         super(tr("Save As..."), "save_as", tr("Save the current data to a new file."),
-                Shortcut.registerShortcut("system:saveas", tr("File: {0}", tr("Save As...")), KeyEvent.VK_S, Shortcut.GROUP_MENU, Shortcut.SHIFT_DEFAULT));
+            Shortcut.registerShortcut("system:saveas", tr("File: {0}", tr("Save As...")),
+            KeyEvent.VK_S, Shortcut.GROUP_MENU+Shortcut.GROUPS_ALT1));
         putValue("help", ht("/Action/SaveAs"));
     }

trunk/src/org/openstreetmap/josm/actions/UnselectAllAction.java

@@ -15 +15 @@
     public UnselectAllAction() {
         super(tr("Unselect All"), "unselectall", tr("Unselect all objects."),
-                Shortcut.registerShortcut("edit:unselectall", tr("Edit: {0}", tr("Unselect All")), KeyEvent.VK_U, Shortcut.GROUP_EDIT), true);
-        // this is not really GROUP_EDIT, but users really would complain if the yhad to reconfigure because we put
-        // the correct group in
-
-        // Add extra shortcut C-S-a
-        Main.registerActionShortcut(this, Shortcut.registerShortcut("edit:unselectallfocus", tr("Edit: {0}", tr("Unselect All (Focus)")),
-                KeyEvent.VK_A, Shortcut.GROUP_MENU, Shortcut.SHIFT_DEFAULT));
-
-
-
-        // Add extra shortcut ESCAPE
-        /*
-         * FIXME: this isn't optimal. In a better world the mapmode actions
-         * would be able to capture keyboard events and react accordingly. But
-         * for now this is a reasonable approximation.
-         */
-        Main.registerActionShortcut(this, Shortcut.registerShortcut("edit:unselectallescape", tr("Edit: {0}", tr("Unselect All (Escape)")),
-                KeyEvent.VK_ESCAPE, Shortcut.GROUP_DIRECT));
+            Shortcut.registerShortcut("edit:unselectall", tr("Edit: {0}",
+            tr("Unselect All")), KeyEvent.VK_ESCAPE, Shortcut.GROUP_DIRECT), true);
 
         putValue("help", ht("/Action/UnselectAll"));

trunk/src/org/openstreetmap/josm/actions/ViewportFollowToggleAction.java

@@ -23 +23 @@
                 "viewport-follow",
                 tr("Enable/disable automatic moving of the map view to last placed node"),
-                Shortcut.registerShortcut("menu:view:viewportfollow", tr("Toggle Viewport Following"),KeyEvent.VK_F, Shortcut.GROUP_MENU, Shortcut.SHIFT_DEFAULT),
+                Shortcut.registerShortcut("menu:view:viewportfollow", tr("Toggle Viewport Following"),
+                KeyEvent.VK_F, Shortcut.GROUP_MENU+Shortcut.GROUPS_ALT1),
                 true /* register shortcut */
         );

trunk/src/org/openstreetmap/josm/gui/dialogs/CommandStackDialog.java

@@ -73 +73 @@
     public CommandStackDialog(final MapFrame mapFrame) {
         super(tr("Command Stack"), "commandstack", tr("Open a list of all commands (undo buffer)."),
-                Shortcut.registerShortcut("subwindow:commandstack", tr("Toggle: {0}", tr("Command Stack")), KeyEvent.VK_O, Shortcut.GROUP_LAYER, Shortcut.SHIFT_DEFAULT), 100, true);
+                Shortcut.registerShortcut("subwindow:commandstack", tr("Toggle: {0}",
+                tr("Command Stack")), KeyEvent.VK_O, Shortcut.GROUP_LAYER
+                +Shortcut.GROUPS_ALT1), 100, true);
         undoTree.addMouseListener(new PopupMenuHandler());
         undoTree.setRootVisible(false);

trunk/src/org/openstreetmap/josm/gui/dialogs/HistoryDialog.java

@@ -64 +64 @@
         super(tr("History"), "history", tr("Display the history of all selected items."),
                 Shortcut.registerShortcut("subwindow:history", tr("Toggle: {0}", tr("History")), KeyEvent.VK_H,
-                        Shortcut.GROUP_LAYER, Shortcut.SHIFT_DEFAULT), 150);
+                        Shortcut.GROUP_LAYER+Shortcut.GROUPS_ALT1), 150);
         build();
         HelpUtil.setHelpContext(this, HelpUtil.ht("/Dialog/History"));