Changeset 4604 in josm for trunk/src/org/openstreetmap/josm


Ignore:
Timestamp:
2011-11-21T22:16:25+01:00 (12 years ago)
Author:
jttt
Message:

Multikey action improvements (see #5515):

  • show text in status bar after shortcut is pressed
  • number layers from 1
  • do not allow Repeat for last layer if layer index conflicts with shortcut
Location:
trunk/src/org/openstreetmap/josm
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/MapStatus.java

    r3783 r4604  
    9595    }
    9696
    97     ImageLabel lonText = new ImageLabel("lon", tr("The geographic longitude at the mouse pointer."), 11);
    98     ImageLabel nameText = new ImageLabel("name", tr("The name of the object at the mouse pointer."), 20);
    99     JTextField helpText = new JTextField();
    100     ImageLabel latText = new ImageLabel("lat", tr("The geographic latitude at the mouse pointer."), 11);
    101     ImageLabel angleText = new ImageLabel("angle", tr("The angle between the previous and the current way segment."), 6);
    102     ImageLabel headingText = new ImageLabel("heading", tr("The (compass) heading of the line segment being drawn."), 6);
    103     ImageLabel distText = new ImageLabel("dist", tr("The length of the new way segment being drawn."), 10);
     97    final ImageLabel lonText = new ImageLabel("lon", tr("The geographic longitude at the mouse pointer."), 11);
     98    final ImageLabel nameText = new ImageLabel("name", tr("The name of the object at the mouse pointer."), 20);
     99    final JTextField helpText = new JTextField();
     100    final ImageLabel latText = new ImageLabel("lat", tr("The geographic latitude at the mouse pointer."), 11);
     101    final ImageLabel angleText = new ImageLabel("angle", tr("The angle between the previous and the current way segment."), 6);
     102    final ImageLabel headingText = new ImageLabel("heading", tr("The (compass) heading of the line segment being drawn."), 6);
     103    final ImageLabel distText = new ImageLabel("dist", tr("The length of the new way segment being drawn."), 10);
    104104
    105105    /**
    … …  
    108108     */
    109109    public Thread thread;
     110
     111    private final List<StatusTextHistory> statusText = new ArrayList<StatusTextHistory>();
     112
     113    private static class StatusTextHistory {
     114        final Object id;
     115        final String text;
     116
     117        public StatusTextHistory(Object id, String text) {
     118            this.id = id;
     119            this.text = text;
     120        }
     121
     122        @Override
     123        public boolean equals(Object obj) {
     124            return obj instanceof StatusTextHistory && ((StatusTextHistory)obj).id == id;
     125        }
     126
     127        @Override
     128        public int hashCode() {
     129            return System.identityHashCode(id);
     130        }
     131    }
    110132
    111133    /**
    … …  
    218240                                        final JLabel lbl = new JLabel(
    219241                                                "<html>"+tr("Middle click again to cycle through.<br>"+
    220                                                 "Hold CTRL to select directly from this list with the mouse.<hr>")+"</html>",
    221                                                 null,
    222                                                 JLabel.HORIZONTAL
    223                                         );
     242                                                        "Hold CTRL to select directly from this list with the mouse.<hr>")+"</html>",
     243                                                        null,
     244                                                        JLabel.HORIZONTAL
     245                                                );
    224246                                        lbl.setHorizontalAlignment(JLabel.LEFT);
    225247                                        c.add(lbl, GBC.eol().insets(2, 0, 2, 0));
    … …  
    472494                    ImageProvider.get(OsmPrimitiveType.from(osm)),
    473495                    JLabel.HORIZONTAL
    474             ) {
     496                    ) {
    475497                // This is necessary so the label updates its colors when the
    476498                // selection is changed from the outside
    … …  
    656678
    657679    public void setHelpText(String t) {
    658         helpText.setText(t);
    659         helpText.setToolTipText(t);
     680        setHelpText(null, t);
     681    }
     682    public void setHelpText(Object id, String text)  {
     683
     684        StatusTextHistory entry = new StatusTextHistory(id, text);
     685
     686        statusText.remove(entry);
     687        statusText.add(entry);
     688
     689        helpText.setText(text);
     690        helpText.setToolTipText(text);
     691    }
     692    public void resetHelpText(Object id) {
     693        if (statusText.isEmpty())
     694            return;
     695
     696        StatusTextHistory entry = new StatusTextHistory(id, null);
     697        if (statusText.get(statusText.size() - 1).equals(entry)) {
     698            if (statusText.size() == 1) {
     699                setHelpText("");
     700            } else {
     701                StatusTextHistory history = statusText.get(statusText.size() - 2);
     702                setHelpText(history.id, history.text);
     703            }
     704        }
     705        statusText.remove(entry);
    660706    }
    661707    public void setAngle(double a) {

  • trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

    r4596 r4604  
    460460
    461461        @Override
    462         public void executeMultikeyAction(int index) {
     462        public void executeMultikeyAction(int index, boolean repeat) {
    463463            Layer l = LayerListDialog.getLayerForIndex(index);
    464464            if (l != null) {
    465465                l.toggleVisible();
    466466                lastLayer = new WeakReference<Layer>(l);
    467             }
    468         }
    469 
    470         @Override
    471         public void repeateLastMultikeyAction() {
    472             if (lastLayer != null) {
    473                 Layer l = lastLayer.get();
     467            } else if (repeat && lastLayer != null) {
     468                l = lastLayer.get();
    474469                if (LayerListDialog.isLayerValid(l)) {
    475470                    l.toggleVisible();
    … …  
    690685
    691686        @Override
    692         public void executeMultikeyAction(int index) {
     687        public void executeMultikeyAction(int index, boolean repeat) {
    693688            Layer l = LayerListDialog.getLayerForIndex(index);
    694689            if (l != null) {
    695690                execute(l);
    696691            }
    697         }
    698 
    699         @Override
    700         public void repeateLastMultikeyAction() {
    701             // Do nothing, repating not supported
    702692        }
    703693
    … …  
    15581548        List<Layer> layers = Main.map.mapView.getAllLayersAsList();
    15591549
    1560         if (index < layers.size())
     1550        if (index < layers.size() && index >= 0)
    15611551            return layers.get(index);
    15621552        else

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r4536 r4604  
    3232import java.util.LinkedList;
    3333import java.util.List;
     34import java.util.Map;
    3435import java.util.Map.Entry;
    35 import java.util.Map;
    3636import java.util.Set;
    3737import java.util.TreeMap;
    … …  
    6767import org.openstreetmap.josm.Main;
    6868import org.openstreetmap.josm.actions.JosmAction;
     69import org.openstreetmap.josm.actions.search.SearchAction.SearchMode;
     70import org.openstreetmap.josm.actions.search.SearchAction.SearchSetting;
    6971import org.openstreetmap.josm.command.ChangeCommand;
    7072import org.openstreetmap.josm.command.ChangePropertyCommand;
    … …  
    7274import org.openstreetmap.josm.command.SequenceCommand;
    7375import org.openstreetmap.josm.data.SelectionChangedListener;
     76import org.openstreetmap.josm.data.osm.DataSet;
    7477import org.openstreetmap.josm.data.osm.IRelation;
    7578import org.openstreetmap.josm.data.osm.Node;
    … …  
    105108import org.openstreetmap.josm.tools.Shortcut;
    106109import org.openstreetmap.josm.tools.Utils;
    107 import org.openstreetmap.josm.actions.search.SearchAction.SearchMode;
    108 import org.openstreetmap.josm.actions.search.SearchAction.SearchSetting;
    109110
    110111/**
    … …  
    170171    // button in the upper right corner of this dialog
    171172    public static JPanel pluginHook = new JPanel();
    172    
     173
    173174    private JPopupMenu propertyMenu;
    174175    private JPopupMenu membershipMenu;
    … …  
    338339                    ed.setButtonIcons(new String[]{"purge", "cancel"});
    339340                    ed.setContent(tr("You changed the key from ''{0}'' to ''{1}''.\n"
    340                     + "The new key is already used, overwrite values?", key, newkey));
     341                            + "The new key is already used, overwrite values?", key, newkey));
    341342                    ed.setCancelButton(2);
    342343                    ed.toggleEnable("overwriteEditKey");
    343344                    ed.showDialog();
    344345
    345                     if (ed.getValue() != 1) {
     346                    if (ed.getValue() != 1)
    346347                        return;
    347                     }
    348348                    break;
    349349                }
    … …  
    389389
    390390    /**
    391      * For a given key k, return a list of keys which are used as keys for 
     391     * For a given key k, return a list of keys which are used as keys for
    392392     * auto-completing values to increase the search space.
    393393     * @param key the key k
    … …  
    395395     */
    396396    static List<String> getAutocompletionKeys(String key) {
    397         if ("name".equals(key) || "addr:street".equals(key)) {
     397        if ("name".equals(key) || "addr:street".equals(key))
    398398            return Arrays.asList("addr:street", "name");
    399         } else {
     399        else
    400400            return Arrays.asList(key);
    401         }
    402401    }
    403402
    … …  
    424423     */
    425424    void add() {
    426         Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
     425        DataSet ds = Main.main.getCurrentDataSet();
     426        if (ds == null) return;
     427        Collection<OsmPrimitive> sel = ds.getSelected();
    427428        if (sel.isEmpty()) return;
    428429
    … …  
    435436        List<AutoCompletionListItem> keyList = autocomplete.getKeys();
    436437
    437         AutoCompletionListItem itemToSelect = null; 
     438        AutoCompletionListItem itemToSelect = null;
    438439        // remove the object's tag keys from the list
    439440        Iterator<AutoCompletionListItem> iter = keyList.iterator();
    … …  
    445446            for (int i = 0; i < propertyData.getRowCount(); ++i) {
    446447                if (item.getValue().equals(propertyData.getValueAt(i, 0))) {
    447                     if (itemToSelect == item)
     448                    if (itemToSelect == item) {
    448449                        itemToSelect = null;
     450                    }
    449451                    iter.remove();
    450452                    break;
    … …  
    468470            keys.setSelectedItem(itemToSelect);
    469471            /* don't add single chars, as they are no properly selected */
    470             if(lastAddValue != null && lastAddValue.length() > 1)
     472            if(lastAddValue != null && lastAddValue.length() > 1) {
    471473                values.setSelectedItem(lastAddValue);
     474            }
    472475        }
    473476
    … …  
    506509        // get the combo box' editor component
    507510        JTextComponent editor = (JTextComponent)values.getEditor()
    508         .getEditorComponent();
     511                .getEditorComponent();
    509512        // Refresh the values model when focus is gained
    510513        FocusAdapter focus = new FocusAdapter() {
    … …  
    632635        propertyMenu.addSeparator();
    633636        propertyMenu.add(helpAction);
    634        
     637
    635638        propertyData.setColumnIdentifiers(new String[]{tr("Key"),tr("Value")});
    636639        propertyTable.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
    … …  
    813816        getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT).put(
    814817                KeyStroke.getKeyStroke(KeyEvent.VK_DELETE, 0),"delete"
    815         );
     818                );
    816819        getActionMap().put("delete", deleteAction);
    817820
    818821        JScrollPane scrollPane = (JScrollPane) createLayout(bothTables, true, Arrays.asList(new SideButton[] {
    819             this.btnAdd, this.btnEdit, this.btnDel
     822                this.btnAdd, this.btnEdit, this.btnDel
    820823        }));
    821824
    … …  
    962965                return comp;
    963966            }}
    964         );
     967                );
    965968
    966969        for (Relation r: sortedRelations) {
    … …  
    10271030            super(tr("Delete"), "dialogs/delete", tr("Delete the selected key in all objects"),
    10281031                    Shortcut.registerShortcut("properties:delete", tr("Delete Properties"), KeyEvent.VK_D,
    1029                     Shortcut.GROUP_MNEMONIC), false);
     1032                            Shortcut.GROUP_MNEMONIC), false);
    10301033            updateEnabledState();
    10311034        }
    … …  
    10981101                    (propertyTable != null && propertyTable.getSelectedRowCount() == 1)
    10991102                    ^ (membershipTable != null && membershipTable.getSelectedRowCount() == 1)
    1100             );
     1103                    );
    11011104        }
    11021105
    … …  
    11111114            super(tr("Add"), "dialogs/add", tr("Add a new key/value pair to all objects"),
    11121115                    Shortcut.registerShortcut("properties:add", tr("Add Property"), KeyEvent.VK_A,
    1113                     Shortcut.GROUP_MNEMONIC), false);
     1116                            Shortcut.GROUP_MNEMONIC), false);
    11141117        }
    11151118
    … …  
    11241127            super(tr("Edit"), "dialogs/edit", tr("Edit the value of the selected key for all objects"),
    11251128                    Shortcut.registerShortcut("properties:edit", tr("Edit Properties"), KeyEvent.VK_S,
    1126                     Shortcut.GROUP_MNEMONIC), false);
     1129                            Shortcut.GROUP_MNEMONIC), false);
    11271130            updateEnabledState();
    11281131        }
    … …  
    11461149                    (propertyTable != null && propertyTable.getSelectedRowCount() == 1)
    11471150                    ^ (membershipTable != null && membershipTable.getSelectedRowCount() == 1)
    1148             );
     1151                    );
    11491152        }
    11501153
    … …  
    11741177                            ((Map<String,Integer>)propertyData.getValueAt(row, 1))
    11751178                            .entrySet().iterator().next().getKey(), "UTF-8"
    1176                     );
     1179                            );
    11771180
    11781181                    uris.add(new URI(String.format("%s%sTag:%s=%s", base, lang, key, val)));
    … …  
    11861189                    String type = URLEncoder.encode(
    11871190                            ((Relation)membershipData.getValueAt(row, 0)).get("type"), "UTF-8"
    1188                     );
     1191                            );
    11891192
    11901193                    if (type != null && !type.equals("")) {
    … …  
    12201223                                            .replace("=", "%3D") /* do not URLencode whole string! */
    12211224                                            .replaceFirst("/wiki/", "/w/index.php?redirect=no&title=")
    1222                                     ).toURL().openConnection();
     1225                                            ).toURL().openConnection();
    12231226                                    conn.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect",15)*1000);
    12241227
    … …  
    12571260        return propertyMenu.add(a);
    12581261    }
    1259    
     1262
    12601263    public void addPropertyPopupMenuListener(PopupMenuListener l) {
    12611264        propertyMenu.addPopupMenuListener(l);
    … …  
    12651268        propertyMenu.addPopupMenuListener(l);
    12661269    }
    1267    
     1270
    12681271    @SuppressWarnings("unchecked")
    12691272    public Tag getSelectedProperty() {
    … …  
    12721275        TreeMap<String, Integer> map = (TreeMap<String, Integer>) propertyData.getValueAt(row, 1);
    12731276        return new Tag(
    1274                 propertyData.getValueAt(row, 0).toString(), 
     1277                propertyData.getValueAt(row, 0).toString(),
    12751278                map.size() > 1 ? "" : map.keySet().iterator().next());
    12761279    }
    1277    
     1280
    12781281    public void addMembershipPopupMenuSeparator() {
    12791282        membershipMenu.addSeparator();
    12801283    }
    1281    
     1284
    12821285    public JMenuItem addMembershipPopupMenuAction(Action a) {
    12831286        return membershipMenu.add(a);
    12841287    }
    1285    
     1288
    12861289    public void addMembershipPopupMenuListener(PopupMenuListener l) {
    12871290        membershipMenu.addPopupMenuListener(l);
    … …  
    12911294        membershipMenu.addPopupMenuListener(l);
    12921295    }
    1293    
     1296
    12941297    public IRelation getSelectedMembershipRelation() {
    12951298        int row = membershipTable.getSelectedRow();
    … …  
    13011304        public void setRelation(Relation relation);
    13021305    }
    1303    
     1306
    13041307    static abstract class AbstractRelationAction extends AbstractAction implements RelationRelated {
    13051308        protected Relation relation;
    … …  
    13111314        }
    13121315    }
    1313    
     1316
    13141317    static class SelectRelationAction extends AbstractRelationAction {
    13151318        boolean selectionmode;
    … …  
    13821385                    buildSetOfIncompleteMembers(relation),
    13831386                    Main.map.mapView.getEditLayer()
    1384             ));
     1387                    ));
    13851388        }
    13861389    }
    … …  
    13921395        @Override
    13931396        public void actionPerformed(ActionEvent ae) {
    1394             if (propertyTable.getSelectedRowCount() != 1) {
     1397            if (propertyTable.getSelectedRowCount() != 1)
    13951398                return;
    1396             }
    13971399            String key = propertyData.getValueAt(propertyTable.getSelectedRow(), 0).toString();
    13981400            Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
    1399             if (sel.isEmpty()) {
     1401            if (sel.isEmpty())
    14001402                return;
    1401             }
    14021403            Set<String> values = new TreeSet<String>();
    14031404            for (OsmPrimitive p : sel) {
    … …  
    14701471
    14711472        public void actionPerformed(ActionEvent e) {
    1472             if (propertyTable.getSelectedRowCount() != 1) {
     1473            if (propertyTable.getSelectedRowCount() != 1)
    14731474                return;
    1474             }
    14751475            String key = propertyData.getValueAt(propertyTable.getSelectedRow(), 0).toString();
    14761476            Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
    1477             if (sel.isEmpty()) {
     1477            if (sel.isEmpty())
    14781478                return;
    1479             }
    14801479            String sep = "";
    14811480            String s = "";

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java

    r4595 r4604  
    460460
    461461        @Override
    462         public void executeMultikeyAction(int index) {
     462        public void executeMultikeyAction(int index, boolean repeat) {
    463463            Layer l = LayerListDialog.getLayerForIndex(index);
    464             if (l != null && l instanceof MarkerLayer) {
    465                 execute((MarkerLayer) l);
    466             }
    467         }
    468 
    469         @Override
    470         public void repeateLastMultikeyAction() {
    471             if (lastLayer != null) {
    472                 MarkerLayer l = lastLayer.get();
     464            if (l != null) {
     465                if (l instanceof MarkerLayer) {
     466                    execute((MarkerLayer) l);
     467                }
     468            } else if (repeat && lastLayer != null) {
     469                l = lastLayer.get();
    473470                if (LayerListDialog.isLayerValid(l)) {
    474                     execute(l);
    475                 }
    476             }
    477         }
     471                    execute((MarkerLayer) l);
     472                }
     473            }
     474        }
     475
    478476
    479477        private void execute(MarkerLayer l) {
    … …  
    516514
    517515        @Override
    518         public void executeMultikeyAction(int index) {
     516        public void executeMultikeyAction(int index, boolean repeat) {
    519517            Layer l = LayerListDialog.getLayerForIndex(index);
    520             if (l != null && l instanceof MarkerLayer) {
    521                 execute((MarkerLayer) l);
    522             }
    523         }
    524 
    525         @Override
    526         public void repeateLastMultikeyAction() {
    527             if (lastLayer != null) {
    528                 MarkerLayer l = lastLayer.get();
     518            if (l != null) {
     519                if (l instanceof MarkerLayer) {
     520                    execute((MarkerLayer) l);
     521                }
     522            } else if (repeat && lastLayer != null) {
     523                l = lastLayer.get();
    529524                if (LayerListDialog.isLayerValid(l)) {
    530                     execute(l);
     525                    execute((MarkerLayer) l);
    531526                }
    532527            }

  • trunk/src/org/openstreetmap/josm/tools/MultikeyActionsHandler.java

    r4595 r4604  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.tools;
     3
     4import static org.openstreetmap.josm.tools.I18n.tr;
    35
    46import java.awt.KeyEventDispatcher;
    … …  
    2123public class MultikeyActionsHandler {
    2224
    23     private static final long DIALOG_DELAY = 2000;
     25    private static final long DIALOG_DELAY = 1000;
     26    private static final String STATUS_BAR_ID = new String("multikeyShortcut");
    2427
    2528    private class MyKeyEventDispatcher implements KeyEventDispatcher {
    … …  
    3134
    3235            if (lastAction != null && e.getID() == KeyEvent.KEY_PRESSED) {
    33                 if (e.getKeyCode() == lastAction.shortcut.getKeyCode()) {
    34                     lastAction.action.repeateLastMultikeyAction();
    35                 } else {
    36                     int index = getIndex(e.getKeyChar());
    37                     if (index >= 0) {
    38                         lastAction.action.executeMultikeyAction(index);
    39                     }
     36                int index = getIndex(e.getKeyCode());
     37                if (index >= 0) {
     38                    lastAction.action.executeMultikeyAction(index, e.getKeyCode() == lastAction.shortcut.getKeyCode());
    4039                }
    4140                lastAction = null;
     41                Main.map.statusLine.resetHelpText(STATUS_BAR_ID);
    4242                return true;
    4343            }
    … …  
    4545        }
    4646
    47         private int getIndex(char lastKey) {
    48             if (lastKey >= KeyEvent.VK_0 && lastKey <= KeyEvent.VK_9)
    49                 return lastKey - KeyEvent.VK_0;
     47        private int getIndex(int lastKey) {
     48            if (lastKey >= KeyEvent.VK_1 && lastKey <= KeyEvent.VK_9)
     49                return lastKey - KeyEvent.VK_1;
     50            else if (lastKey == KeyEvent.VK_0)
     51                return 9;
    5052            else if (lastKey >= KeyEvent.VK_A && lastKey <= KeyEvent.VK_Z)
    5153                return lastKey - KeyEvent.VK_A + 10;
    … …  
    7072            lastAction = this;
    7173            timer.schedule(new MyTimerTask(lastTimestamp, lastAction), DIALOG_DELAY);
     74            Main.map.statusLine.setHelpText(STATUS_BAR_ID, tr("{0}... [please type its number]", (String) action.getValue(SHORT_DESCRIPTION)));
    7275        }
    7376
    … …  
    132135        layers.add(lbTitle);
    133136
     137        char repeatKey = (char) action.shortcut.getKeyCode();
     138        boolean repeatKeyUsed = false;
     139
    134140
    135141        for (final MultikeyInfo info: action.action.getMultikeyCombinations()) {
     142
     143            if (info.getShortcut() == repeatKey) {
     144                repeatKeyUsed = true;
     145            }
     146
    136147            JMenuItem item = new JMenuItem(formatMenuText(action.shortcut, String.valueOf(info.getShortcut()), info.getDescription()));
    137148            item.setMnemonic(info.getShortcut());
    … …  
    139150                @Override
    140151                public void actionPerformed(ActionEvent e) {
    141                     action.action.executeMultikeyAction(info.getIndex());
     152                    Main.map.statusLine.resetHelpText(STATUS_BAR_ID);
     153                    action.action.executeMultikeyAction(info.getIndex(), false);
    142154                }
    143155            });
    … …  
    145157        }
    146158
    147         MultikeyInfo lastLayer = action.action.getLastMultikeyAction();
    148         if (lastLayer != null) {
    149             JMenuItem repeateItem = new JMenuItem(formatMenuText(action.shortcut,
    150                     KeyEvent.getKeyText(action.shortcut.getKeyCode()),
    151                     "Repeat " + lastLayer.getDescription()));
    152             repeateItem.addActionListener(new ActionListener() {
    153                 @Override
    154                 public void actionPerformed(ActionEvent e) {
    155                     action.action.repeateLastMultikeyAction();
    156                 }
    157             });
    158             layers.add(repeateItem);
     159        if (!repeatKeyUsed) {
     160            MultikeyInfo lastLayer = action.action.getLastMultikeyAction();
     161            if (lastLayer != null) {
     162                JMenuItem repeateItem = new JMenuItem(formatMenuText(action.shortcut,
     163                        KeyEvent.getKeyText(action.shortcut.getKeyCode()),
     164                        "Repeat " + lastLayer.getDescription()));
     165                repeateItem.setMnemonic(action.shortcut.getKeyCode());
     166                repeateItem.addActionListener(new ActionListener() {
     167                    @Override
     168                    public void actionPerformed(ActionEvent e) {
     169                        Main.map.statusLine.resetHelpText(STATUS_BAR_ID);
     170                        action.action.executeMultikeyAction(-1, true);
     171                    }
     172                });
     173                layers.add(repeateItem);
     174            }
    159175        }
    160176

  • trunk/src/org/openstreetmap/josm/tools/MultikeyShortcutAction.java

    r4595 r4604  
    2222
    2323        public char getShortcut() {
    24             if (index < 10)
    25                 return (char)('0' + index);
     24            if (index < 9)
     25                return (char)('1' + index);
     26            else if (index == 9)
     27                return '0';
    2628            else
    2729                return (char)('A' +  index - 10);
    … …  
    3335    }
    3436
    35     void executeMultikeyAction(int index);
    36     void repeateLastMultikeyAction();
     37    void executeMultikeyAction(int index, boolean repeatLastAction);
    3738    List<MultikeyInfo> getMultikeyCombinations();
    3839    MultikeyInfo getLastMultikeyAction();
Note: See TracChangeset for help on using the changeset viewer.