Changeset 9320 in josm for trunk/src/org


Ignore:
Timestamp:
2016-01-05T15:34:45+01:00 (4 years ago)
Author:
simon04
Message:

Refactoring (make Help a JosmAction)

Location:
trunk/src/org/openstreetmap/josm
Files:
3 edited

Legend:

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

    r8510 r9320  
    77import java.awt.Point;
    88import java.awt.event.ActionEvent;
     9import java.awt.event.KeyEvent;
    910
    10 import javax.swing.AbstractAction;
    1111import javax.swing.SwingUtilities;
    1212
     
    1515import org.openstreetmap.josm.gui.help.HelpUtil;
    1616import org.openstreetmap.josm.io.OnlineResource;
    17 import org.openstreetmap.josm.tools.ImageProvider;
     17import org.openstreetmap.josm.tools.Shortcut;
    1818
    1919/**
     
    2121 * @since 155
    2222 */
    23 public class HelpAction extends AbstractAction {
     23public class HelpAction extends JosmAction {
    2424
    2525    /**
     
    2727     */
    2828    public HelpAction() {
    29         super(tr("Help"));
    30         new ImageProvider("help").getResource().getImageIcon(this);
    31         putValue("toolbar", "help");
     29        this(true);
     30    }
     31
     32    private HelpAction(boolean shortcut) {
     33        super(tr("Help"), "help", null,
     34                shortcut ? Shortcut.registerShortcut("system:help", tr("Help"), KeyEvent.VK_F1, Shortcut.DIRECT) : null,
     35                true);
    3236        setEnabled(!Main.isOffline(OnlineResource.JOSM_WEBSITE));
     37    }
     38
     39    /**
     40     * Constructs a new {@code HelpAction} without assigning a shortcut.
     41     * @return a new {@code HelpAction}
     42     */
     43    public static HelpAction createWithoutShortcut() {
     44        return new HelpAction(false);
    3345    }
    3446
  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r9261 r9320  
    818818        });
    819819
    820         helpMenu.add(statusreport);
    821         helpMenu.add(reportbug);
     820        add(helpMenu, statusreport);
     821        add(helpMenu, reportbug);
    822822        helpMenu.addSeparator();
    823823
    824         // FIXME why is help not a JosmAction?
    825         helpMenu.add(help).setAccelerator(Shortcut.registerShortcut("system:help", tr("Help"), KeyEvent.VK_F1,
    826                 Shortcut.DIRECT).getKeyStroke());
     824        add(helpMenu, help);
    827825        add(helpMenu, about);
    828826        add(Box.createHorizontalGlue());
  • trunk/src/org/openstreetmap/josm/gui/help/HelpUtil.java

    r8846 r9320  
    192192            return Main.main.menu.help;
    193193        }
    194         return new HelpAction();
     194        return HelpAction.createWithoutShortcut();
    195195    }
    196196
Note: See TracChangeset for help on using the changeset viewer.