Changeset 4139 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2011-06-18T15:26:52+02:00 (14 years ago)

Author:

stoecker

Message:

fix #6474 - fix toolbar action entries for some actions and fix fullscreen mode start

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• FullscreenToggleAction.java (modified) (2 diffs)
• HelpAction.java (modified) (1 diff)
• HistoryInfoAction.java (modified) (2 diffs)
• InfoAction.java (modified) (2 diffs)
• JosmAction.java (modified) (1 diff)
• JumpToAction.java (modified) (1 diff)
• MergeLayerAction.java (modified) (1 diff)
• MoveAction.java (modified) (1 diff)
• OrthogonalizeAction.java (modified) (1 diff)
• ShowStatusReportAction.java (modified) (1 diff)
• WireframeToggleAction.java (modified) (1 diff)

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

@@ -37 +37 @@
                 tr("Toggle fullscreen view"),
                 Shortcut.registerShortcut("menu:view:fullscreen", tr("Toggle fullscreen view"),KeyEvent.VK_F11, Shortcut.GROUP_DIRECT),
-                true /* register shortcut */
+                false /* register */
         );
         putValue("help", ht("/Action/FullscreenView"));
+        putValue("toolbar", "fullscreen");
+        Main.toolbar.register(this);
         gd = GraphicsEnvironment.getLocalGraphicsEnvironment().getDefaultScreenDevice();
         selected = Main.pref.getBoolean("draw.fullscreen", false);
@@ -77 +79 @@
         Main.pref.put("draw.fullscreen", selected);
         notifySelectedState();
+        setMode();
+    }
 
+    public void initial() {
+        if(selected)
+            setMode();
+    }
+
+    protected void setMode() {
         Frame frame = (Frame) Main.parent;
 

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

@@ -24 +24 @@
     public HelpAction() {
         super(tr("Help"), ImageProvider.get("help"));
+        putValue("toolbar", "help");
     }
 

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

@@ -6 +6 @@
 import java.awt.event.KeyEvent;
 
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
@@ -17 +18 @@
                 tr("Display history information about OSM ways, nodes, or relations."),
                 Shortcut.registerShortcut("core:history",
-                        tr("Object history"), KeyEvent.VK_H, Shortcut.GROUP_HOTKEY), true);
+                        tr("Object history"), KeyEvent.VK_H, Shortcut.GROUP_HOTKEY), false);
         putValue("help", ht("/Action/ObjectHistory"));
+        putValue("toolbar", "action/historyinfo");
+        Main.toolbar.register(this);
     }
 

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

@@ -7 +7 @@
 import java.awt.event.KeyEvent;
 
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
@@ -17 +18 @@
                 tr("Display object information about OSM nodes, ways, or relations."),
                 Shortcut.registerShortcut("core:information",
-                        tr("Info about Element"), KeyEvent.VK_I, Shortcut.GROUP_HOTKEY), true);
+                        tr("Info about Element"), KeyEvent.VK_I, Shortcut.GROUP_HOTKEY), false);
         putValue("help", ht("/Action/InfoAboutElements"));
+        putValue("toolbar", "action/info");
+        Main.toolbar.register(this);
     }
 

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

@@ -86 +86 @@
         }
         putValue(SHORT_DESCRIPTION, Main.platform.makeTooltip(tooltip, sc));
-        putValue("toolbar", iconName);
+        if (getValue("toolbar") == null) {
+            putValue("toolbar", iconName);
+        }
         if (register) {
             Main.toolbar.register(this);

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

@@ -30 +30 @@
     public JumpToAction() {
         super(tr("Jump To Position"), null, tr("Opens a dialog that allows to jump to a specific location"), Shortcut.registerShortcut("tools:jumpto", tr("Tool: {0}", tr("Jump To Position")),
-        KeyEvent.VK_G, Shortcut.GROUP_HOTKEY), true);
+        KeyEvent.VK_G, Shortcut.GROUP_HOTKEY), false);
+        putValue("toolbar", "action/jumpto");
+        Main.toolbar.register(this);
     }
 

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

@@ -19 +19 @@
         super(tr("Merge layer"), "dialogs/mergedown", tr("Merge the current layer into another layer"), Shortcut
                 .registerShortcut("system:merge", tr("Edit: {0}", tr("Merge")), KeyEvent.VK_M, Shortcut.GROUP_MENU),
-                true /* register */
+                false /* register */
         );
         putValue("help", ht("/Action/MergeLayer"));
+        putValue("toolbar", "action/mergelayer");
+        Main.toolbar.register(this);
     }
 

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

@@ -32 +32 @@
 
     // any better idea?
-    private static Object calltosupermustbefirststatementinconstructor(Direction dir, boolean text) {
-        Shortcut sc;
+    private static String calltosupermustbefirststatementinconstructor_text(Direction dir) {
         String directiontext;
         if        (dir == Direction.UP)   {
             directiontext = tr("up");
-            sc = Shortcut.registerShortcut("core:moveup",    tr("Move objects {0}", directiontext), KeyEvent.VK_UP,    Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
         } else if (dir == Direction.DOWN)  {
             directiontext = tr("down");
-            sc = Shortcut.registerShortcut("core:movedown",  tr("Move objects {0}", directiontext), KeyEvent.VK_DOWN,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
         } else if (dir == Direction.LEFT)  {
             directiontext = tr("left");
-            sc = Shortcut.registerShortcut("core:moveleft",  tr("Move objects {0}", directiontext), KeyEvent.VK_LEFT,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
         } else { //dir == Direction.RIGHT) {
             directiontext = tr("right");
-            sc = Shortcut.registerShortcut("core:moveright", tr("Move objects {0}", directiontext), KeyEvent.VK_RIGHT, Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
         }
-        if (text)
-            return directiontext;
-        else
-            return sc;
+        return directiontext;
+    }
+
+    // any better idea?
+    private static Shortcut calltosupermustbefirststatementinconstructor(Direction dir) {
+        Shortcut sc;
+        if        (dir == Direction.UP)   {
+            sc = Shortcut.registerShortcut("core:moveup",    tr("Move objects {0}", tr("up")), KeyEvent.VK_UP,    Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
+        } else if (dir == Direction.DOWN)  {
+            sc = Shortcut.registerShortcut("core:movedown",  tr("Move objects {0}", tr("down")), KeyEvent.VK_DOWN,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
+        } else if (dir == Direction.LEFT)  {
+            sc = Shortcut.registerShortcut("core:moveleft",  tr("Move objects {0}", tr("left")), KeyEvent.VK_LEFT,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
+        } else { //dir == Direction.RIGHT) {
+            sc = Shortcut.registerShortcut("core:moveright", tr("Move objects {0}", tr("right")), KeyEvent.VK_RIGHT, Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
+        }
+        return sc;
     }
 
     public MoveAction(Direction dir) {
-        super(tr("Move {0}", calltosupermustbefirststatementinconstructor(dir, true)), null,
-                tr("Moves Objects {0}", calltosupermustbefirststatementinconstructor(dir, true)),
-                (Shortcut)calltosupermustbefirststatementinconstructor(dir, false), true);
+        super(tr("Move {0}", calltosupermustbefirststatementinconstructor_text(dir)), null,
+                tr("Moves Objects {0}", calltosupermustbefirststatementinconstructor_text(dir)),
+                calltosupermustbefirststatementinconstructor(dir), false);
         myDirection = dir;
         putValue("help", ht("/Action/Move"));
+        if        (dir == Direction.UP)   {
+            putValue("toolbar", "action/move/up");
+        } else if (dir == Direction.DOWN)  {
+            putValue("toolbar", "action/move/down");
+        } else if (dir == Direction.LEFT)  {
+            putValue("toolbar", "action/move/left");
+        } else { //dir == Direction.RIGHT) {
+            putValue("toolbar", "action/move/right");
+        }
+        Main.toolbar.register(this);
     }
 

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

@@ -81 +81 @@
                     Shortcut.registerShortcut("tools:orthogonalizeUndo", tr("Tool: {0}", tr("Orthogonalize Shape / Undo")),
                             KeyEvent.VK_Q,
-                            Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true);
+                            Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), false);
+            putValue("toolbar", "action/orthogonalize/undo");
+            Main.toolbar.register(this);
         }
         public void actionPerformed(ActionEvent e) {

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

@@ -44 +44 @@
 
         putValue("help", ht("/Action/ShowStatusReport"));
+        putValue("toolbar", "help/showstatusreport");
     }
 

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

@@ -20 +20 @@
                 tr("Enable/disable rendering the map as wireframe only"),
                 Shortcut.registerShortcut("menu:view:wireframe", tr("Toggle Wireframe view"),KeyEvent.VK_W, Shortcut.GROUP_MENU),
-                true /* register shortcut */
+                false /* register toolbar */
         );
+        putValue("toolbar", "wireframe");
+        Main.toolbar.register(this);
         putValue(SELECTED_KEY, MapRendererFactory.getInstance().isWireframeMapRendererActive());
     }