Changeset 4139 in josm


Ignore:
Timestamp:
2011-06-18T15:26:52+02:00 (10 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
Files:
14 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/FullscreenToggleAction.java

    r4045 r4139  
    3737                tr("Toggle fullscreen view"),
    3838                Shortcut.registerShortcut("menu:view:fullscreen", tr("Toggle fullscreen view"),KeyEvent.VK_F11, Shortcut.GROUP_DIRECT),
    39                 true /* register shortcut */
     39                false /* register */
    4040        );
    4141        putValue("help", ht("/Action/FullscreenView"));
     42        putValue("toolbar", "fullscreen");
     43        Main.toolbar.register(this);
    4244        gd = GraphicsEnvironment.getLocalGraphicsEnvironment().getDefaultScreenDevice();
    4345        selected = Main.pref.getBoolean("draw.fullscreen", false);
     
    7779        Main.pref.put("draw.fullscreen", selected);
    7880        notifySelectedState();
     81        setMode();
     82    }
    7983
     84    public void initial() {
     85        if(selected)
     86            setMode();
     87    }
     88
     89    protected void setMode() {
    8090        Frame frame = (Frame) Main.parent;
    8191
  • trunk/src/org/openstreetmap/josm/actions/HelpAction.java

    r2990 r4139  
    2424    public HelpAction() {
    2525        super(tr("Help"), ImageProvider.get("help"));
     26        putValue("toolbar", "help");
    2627    }
    2728
  • trunk/src/org/openstreetmap/josm/actions/HistoryInfoAction.java

    r3759 r4139  
    66import java.awt.event.KeyEvent;
    77
     8import org.openstreetmap.josm.Main;
    89import org.openstreetmap.josm.data.osm.OsmPrimitive;
    910import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
     
    1718                tr("Display history information about OSM ways, nodes, or relations."),
    1819                Shortcut.registerShortcut("core:history",
    19                         tr("Object history"), KeyEvent.VK_H, Shortcut.GROUP_HOTKEY), true);
     20                        tr("Object history"), KeyEvent.VK_H, Shortcut.GROUP_HOTKEY), false);
    2021        putValue("help", ht("/Action/ObjectHistory"));
     22        putValue("toolbar", "action/historyinfo");
     23        Main.toolbar.register(this);
    2124    }
    2225
  • trunk/src/org/openstreetmap/josm/actions/InfoAction.java

    r3761 r4139  
    77import java.awt.event.KeyEvent;
    88
     9import org.openstreetmap.josm.Main;
    910import org.openstreetmap.josm.data.osm.OsmPrimitive;
    1011import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
     
    1718                tr("Display object information about OSM nodes, ways, or relations."),
    1819                Shortcut.registerShortcut("core:information",
    19                         tr("Info about Element"), KeyEvent.VK_I, Shortcut.GROUP_HOTKEY), true);
     20                        tr("Info about Element"), KeyEvent.VK_I, Shortcut.GROUP_HOTKEY), false);
    2021        putValue("help", ht("/Action/InfoAboutElements"));
     22        putValue("toolbar", "action/info");
     23        Main.toolbar.register(this);
    2124    }
    2225
  • trunk/src/org/openstreetmap/josm/actions/JosmAction.java

    r3835 r4139  
    8686        }
    8787        putValue(SHORT_DESCRIPTION, Main.platform.makeTooltip(tooltip, sc));
    88         putValue("toolbar", iconName);
     88        if (getValue("toolbar") == null) {
     89            putValue("toolbar", iconName);
     90        }
    8991        if (register) {
    9092            Main.toolbar.register(this);
  • trunk/src/org/openstreetmap/josm/actions/JumpToAction.java

    r3083 r4139  
    3030    public JumpToAction() {
    3131        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")),
    32         KeyEvent.VK_G, Shortcut.GROUP_HOTKEY), true);
     32        KeyEvent.VK_G, Shortcut.GROUP_HOTKEY), false);
     33        putValue("toolbar", "action/jumpto");
     34        Main.toolbar.register(this);
    3335    }
    3436
  • trunk/src/org/openstreetmap/josm/actions/MergeLayerAction.java

    r3083 r4139  
    1919        super(tr("Merge layer"), "dialogs/mergedown", tr("Merge the current layer into another layer"), Shortcut
    2020                .registerShortcut("system:merge", tr("Edit: {0}", tr("Merge")), KeyEvent.VK_M, Shortcut.GROUP_MENU),
    21                 true /* register */
     21                false /* register */
    2222        );
    2323        putValue("help", ht("/Action/MergeLayer"));
     24        putValue("toolbar", "action/mergelayer");
     25        Main.toolbar.register(this);
    2426    }
    2527
  • trunk/src/org/openstreetmap/josm/actions/MoveAction.java

    r3327 r4139  
    3232
    3333    // any better idea?
    34     private static Object calltosupermustbefirststatementinconstructor(Direction dir, boolean text) {
    35         Shortcut sc;
     34    private static String calltosupermustbefirststatementinconstructor_text(Direction dir) {
    3635        String directiontext;
    3736        if        (dir == Direction.UP)   {
    3837            directiontext = tr("up");
    39             sc = Shortcut.registerShortcut("core:moveup",    tr("Move objects {0}", directiontext), KeyEvent.VK_UP,    Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
    4038        } else if (dir == Direction.DOWN)  {
    4139            directiontext = tr("down");
    42             sc = Shortcut.registerShortcut("core:movedown",  tr("Move objects {0}", directiontext), KeyEvent.VK_DOWN,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
    4340        } else if (dir == Direction.LEFT)  {
    4441            directiontext = tr("left");
    45             sc = Shortcut.registerShortcut("core:moveleft",  tr("Move objects {0}", directiontext), KeyEvent.VK_LEFT,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
    4642        } else { //dir == Direction.RIGHT) {
    4743            directiontext = tr("right");
    48             sc = Shortcut.registerShortcut("core:moveright", tr("Move objects {0}", directiontext), KeyEvent.VK_RIGHT, Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
    4944        }
    50         if (text)
    51             return directiontext;
    52         else
    53             return sc;
     45        return directiontext;
     46    }
     47
     48    // any better idea?
     49    private static Shortcut calltosupermustbefirststatementinconstructor(Direction dir) {
     50        Shortcut sc;
     51        if        (dir == Direction.UP)   {
     52            sc = Shortcut.registerShortcut("core:moveup",    tr("Move objects {0}", tr("up")), KeyEvent.VK_UP,    Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
     53        } else if (dir == Direction.DOWN)  {
     54            sc = Shortcut.registerShortcut("core:movedown",  tr("Move objects {0}", tr("down")), KeyEvent.VK_DOWN,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
     55        } else if (dir == Direction.LEFT)  {
     56            sc = Shortcut.registerShortcut("core:moveleft",  tr("Move objects {0}", tr("left")), KeyEvent.VK_LEFT,  Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
     57        } else { //dir == Direction.RIGHT) {
     58            sc = Shortcut.registerShortcut("core:moveright", tr("Move objects {0}", tr("right")), KeyEvent.VK_RIGHT, Shortcut.GROUPS_ALT1+Shortcut.GROUP_DIRECT);
     59        }
     60        return sc;
    5461    }
    5562
    5663    public MoveAction(Direction dir) {
    57         super(tr("Move {0}", calltosupermustbefirststatementinconstructor(dir, true)), null,
    58                 tr("Moves Objects {0}", calltosupermustbefirststatementinconstructor(dir, true)),
    59                 (Shortcut)calltosupermustbefirststatementinconstructor(dir, false), true);
     64        super(tr("Move {0}", calltosupermustbefirststatementinconstructor_text(dir)), null,
     65                tr("Moves Objects {0}", calltosupermustbefirststatementinconstructor_text(dir)),
     66                calltosupermustbefirststatementinconstructor(dir), false);
    6067        myDirection = dir;
    6168        putValue("help", ht("/Action/Move"));
     69        if        (dir == Direction.UP)   {
     70            putValue("toolbar", "action/move/up");
     71        } else if (dir == Direction.DOWN)  {
     72            putValue("toolbar", "action/move/down");
     73        } else if (dir == Direction.LEFT)  {
     74            putValue("toolbar", "action/move/left");
     75        } else { //dir == Direction.RIGHT) {
     76            putValue("toolbar", "action/move/right");
     77        }
     78        Main.toolbar.register(this);
    6279    }
    6380
  • trunk/src/org/openstreetmap/josm/actions/OrthogonalizeAction.java

    r4126 r4139  
    8181                    Shortcut.registerShortcut("tools:orthogonalizeUndo", tr("Tool: {0}", tr("Orthogonalize Shape / Undo")),
    8282                            KeyEvent.VK_Q,
    83                             Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true);
     83                            Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), false);
     84            putValue("toolbar", "action/orthogonalize/undo");
     85            Main.toolbar.register(this);
    8486        }
    8587        public void actionPerformed(ActionEvent e) {
  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r3299 r4139  
    4444
    4545        putValue("help", ht("/Action/ShowStatusReport"));
     46        putValue("toolbar", "help/showstatusreport");
    4647    }
    4748
  • trunk/src/org/openstreetmap/josm/actions/WireframeToggleAction.java

    r4087 r4139  
    2020                tr("Enable/disable rendering the map as wireframe only"),
    2121                Shortcut.registerShortcut("menu:view:wireframe", tr("Toggle Wireframe view"),KeyEvent.VK_W, Shortcut.GROUP_MENU),
    22                 true /* register shortcut */
     22                false /* register toolbar */
    2323        );
     24        putValue("toolbar", "wireframe");
     25        Main.toolbar.register(this);
    2426        putValue(SELECTED_KEY, MapRendererFactory.getInstance().isWireframeMapRendererActive());
    2527    }
  • trunk/src/org/openstreetmap/josm/gui/MainApplication.java

    r4083 r4139  
    252252            // Main.debug("Main window not maximized");
    253253        }
     254        if(main.menu.fullscreenToggleAction != null)
     255            main.menu.fullscreenToggleAction.initial();
    254256
    255257        SwingUtilities.invokeLater(new Runnable() {
  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r4087 r4139  
    196196
    197197    public final TaggingPresetSearchAction presetSearchAction = new TaggingPresetSearchAction();
    198 
     198    public FullscreenToggleAction fullscreenToggleAction = null;
    199199    /**
    200200     * Add a JosmAction to a menu.
     
    304304        if(!Main.applet) {
    305305            // -- fullscreen toggle action
    306             FullscreenToggleAction fullscreenToggleAction = new FullscreenToggleAction();
     306            fullscreenToggleAction = new FullscreenToggleAction();
    307307            if (fullscreenToggleAction.canFullscreen()) {
    308308                final JCheckBoxMenuItem fullscreen = new JCheckBoxMenuItem(fullscreenToggleAction);
  • trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

    r4033 r4139  
    243243            result.setLength(0);
    244244
    245             escape((String) action.getAction().getValue("toolbar"));
     245            String val = (String) action.getAction().getValue("toolbar");
     246            if(val == null)
     247                return null;
     248            escape(val);
    246249            if (action.getAction() instanceof ParameterizedAction) {
    247250                result.append('(');
     
    753756                    t.add("|");
    754757                } else {
    755                     t.add(parser.saveAction(action));
     758                    String res = parser.saveAction(action);
     759                    if(res != null)
     760                        t.add(res);
    756761                }
    757762            }
     
    783788                    Action action = menuItem.getAction();
    784789                    userObject = action;
    785                     actions.put((String) action.getValue("toolbar"), action);
     790                    String toolbar = (String) action.getValue("toolbar");
     791                    if(toolbar == null) {
     792                        System.out.println(tr("Toolbar action without name: {0}",
     793                        action.getClass().getName()));
     794                    } else {
     795                        Action r = actions.get(toolbar);
     796                        if(r != null &&  r != action) {
     797                            System.out.println(tr("Toolbar action {0} overwritten: {1} gets {2}",
     798                            toolbar, r.getClass().getName(), action.getClass().getName()));
     799                        }
     800                    }
     801                    actions.put(toolbar, action);
    786802                } else {
    787803                    userObject = menuItem.getText();
     
    862878     */
    863879    public Action register(Action action) {
    864         regactions.put((String) action.getValue("toolbar"), action);
     880        String toolbar = (String) action.getValue("toolbar");
     881        if(toolbar == null) {
     882            System.out.println(tr("Registered toolbar action without name: {0}",
     883            action.getClass().getName()));
     884        } else {
     885            Action r = regactions.get(toolbar);
     886            if(r != null) {
     887                System.out.println(tr("Registered toolbar action {0} overwritten: {1} gets {2}",
     888                toolbar, r.getClass().getName(), action.getClass().getName()));
     889            }
     890        }
     891        regactions.put(toolbar, action);
    865892        return action;
    866893    }
Note: See TracChangeset for help on using the changeset viewer.