Ignore:
Timestamp:
2010-06-09T22:14:06+02:00 (14 years ago)
Author:
jttt
Message:

Fix some of "Keystroke %s is already assigned to %s, will be overridden by %s" warnings

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
8 edited

Legend:

Unmodified
Added
Removed

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

    r3266 r3327  
    4141import org.openstreetmap.josm.actions.JoinNodeWayAction;
    4242import org.openstreetmap.josm.actions.JosmAction;
     43import org.openstreetmap.josm.actions.JumpToAction;
    4344import org.openstreetmap.josm.actions.MergeLayerAction;
    4445import org.openstreetmap.josm.actions.MergeNodesAction;
    4546import org.openstreetmap.josm.actions.MergeSelectionAction;
    4647import org.openstreetmap.josm.actions.MirrorAction;
     48import org.openstreetmap.josm.actions.MoveAction;
    4749import org.openstreetmap.josm.actions.MoveNodeAction;
    4850import org.openstreetmap.josm.actions.NewAction;
    … …  
    175177    public final int defaultMenuPos = 5;
    176178
     179    public final JosmAction moveUpAction = new MoveAction(MoveAction.Direction.UP);
     180    public final JosmAction moveDownAction = new MoveAction(MoveAction.Direction.DOWN);
     181    public final JosmAction moveLeftAction = new MoveAction(MoveAction.Direction.LEFT);
     182    public final JosmAction moveRightAction = new MoveAction(MoveAction.Direction.RIGHT);
     183    public final JumpToAction jumpToAct = new JumpToAction();
     184
     185
    177186    /**
    178187     * Add a JosmAction to a menu.

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

    r3177 r3327  
    1717import java.awt.Toolkit;
    1818import java.awt.event.AWTEventListener;
    19 import java.awt.event.ComponentEvent;
    2019import java.awt.event.InputEvent;
    2120import java.awt.event.KeyAdapter;
    … …  
    3938
    4039import org.openstreetmap.josm.Main;
    41 import org.openstreetmap.josm.actions.JumpToAction;
    4240import org.openstreetmap.josm.data.coor.CoordinateFormat;
    4341import org.openstreetmap.josm.data.coor.LatLon;
    … …  
    504502    private AWTEventListener awtListener = new AWTEventListener() {
    505503        public void eventDispatched(AWTEvent event) {
    506             if (event instanceof ComponentEvent &&
    507                     ((ComponentEvent)event).getComponent() == mv) {
     504            if (event instanceof InputEvent &&
     505                    ((InputEvent)event).getComponent() == mv) {
    508506                synchronized (collector) {
    509507                    mouseState.modifiers = ((InputEvent)event).getModifiersEx();
    … …  
    575573        this.collector = new Collector(mapFrame);
    576574
    577         JumpToAction JumpToAct = new JumpToAction();
    578         lonText.addMouseListener(JumpToAct);
    579         latText.addMouseListener(JumpToAct);
     575        lonText.addMouseListener(Main.main.menu.jumpToAct);
     576        latText.addMouseListener(Main.main.menu.jumpToAct);
    580577
    581578        // Listen for mouse movements and set the position text field

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

    r3252 r3327  
    3434import org.openstreetmap.josm.Main;
    3535import org.openstreetmap.josm.actions.AutoScaleAction;
    36 import org.openstreetmap.josm.actions.MoveAction;
    3736import org.openstreetmap.josm.actions.mapmode.MapMode;
    3837import org.openstreetmap.josm.data.Bounds;
    … …  
    190189    public MapView(final JPanel contentPane) {
    191190        Main.pref.addPreferenceChangeListener(this);
     191
     192        //        new MoveAction(MoveAction.Direction.UP);
     193        //        new MoveAction(MoveAction.Direction.DOWN);
     194        //        new MoveAction(MoveAction.Direction.LEFT);
     195        //        new MoveAction(MoveAction.Direction.RIGHT);
     196
    192197        addComponentListener(new ComponentAdapter(){
    193198            @Override public void componentResized(ComponentEvent e) {
    … …  
    206211                    if (!zoomToDataSetBoundingBox(layer.data)) {
    207212                        // no bounding box defined
    208                         new AutoScaleAction("data").actionPerformed(null);
     213                        AutoScaleAction.autoScale("data");
    209214                    }
    210215                } else {
    211                     new AutoScaleAction("layer").actionPerformed(null);
     216                    AutoScaleAction.autoScale("layer");
    212217                }
    213218
    214219                new MapMover(MapView.this, contentPane);
    215                 new MoveAction(MoveAction.Direction.UP);
    216                 new MoveAction(MoveAction.Direction.DOWN);
    217                 new MoveAction(MoveAction.Direction.LEFT);
    218                 new MoveAction(MoveAction.Direction.RIGHT);
    219220            }
    220221        });

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

    r3290 r3327  
    294294     * Launches the search dialog
    295295     */
    296     class SearchAction extends AbstractAction implements EditLayerChangeListener {
     296    static class SearchAction extends AbstractAction implements EditLayerChangeListener {
    297297        public SearchAction() {
    298298            putValue(NAME, tr("Search"));
    … …  
    358358
    359359        public void actionPerformed(ActionEvent e) {
    360             new AutoScaleAction("selection").autoScale();
     360            AutoScaleAction.autoScale("selection");
    361361        }
    362362
    … …  
    423423        }
    424424
     425        @Override
    425426        public void actionPerformed(ActionEvent e) {
    426427            Relation relation = (Relation)model.getSelected().toArray()[0];
    … …  
    428429            Collection<OsmPrimitive> selection = model.getAllElements();
    429430            for (RelationMember member: relation.getMembers()) {
    430                 if (selection.contains(member.getMember()))
     431                if (selection.contains(member.getMember())) {
    431432                    members.add(member);
     433                }
    432434            }
    433435            Main.map.relationListDialog.selectRelation(relation);
    434436            RelationEditor.getEditor(Main.map.mapView.getEditLayer(), relation,
    435                 members).setVisible(true);
     437                    members).setVisible(true);
    436438        }
    437439    }
    … …  
    801803            {
    802804                text.append(": ");
    803                 for(OsmPrimitive o : sel)
    804                    text.append(o.getDisplayName(df));
     805                for(OsmPrimitive o : sel) {
     806                    text.append(o.getDisplayName(df));
     807                }
    805808                setText(text.toString());
    806             }
    807             else
     809            } else {
    808810                setText(tr("Selection: {0}", text));
     811            }
    809812            addActionListener(this);
    810813        }

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

    r3306 r3327  
    5151import javax.swing.table.DefaultTableCellRenderer;
    5252import javax.swing.table.DefaultTableModel;
     53import javax.swing.table.TableColumnModel;
    5354import javax.swing.table.TableModel;
    54 import javax.swing.table.TableColumnModel;
    5555import javax.swing.text.JTextComponent;
    5656
    … …  
    157157
    158158    private DataSetListenerAdapter dataChangedAdapter = new DataSetListenerAdapter(this);
     159    private AddAction addAction = new AddAction();
     160    private Shortcut addActionShortcut = Shortcut.registerShortcut("properties:add", tr("Add Properties"), KeyEvent.VK_B,
     161            Shortcut.GROUP_MNEMONIC);
    159162
    160163    @Override
    … …  
    164167        MapView.addEditLayerChangeListener(this);
    165168        updateSelection();
     169        Main.registerActionShortcut(addAction, addActionShortcut);
    166170    }
    167171
    … …  
    171175        SelectionEventManager.getInstance().removeSelectionListener(this);
    172176        MapView.removeEditLayerChangeListener(this);
     177        Main.unregisterActionShortcut(addActionShortcut);
    173178    }
    174179
    … …  
    331336     * @param row
    332337     */
    333     @SuppressWarnings("unchecked")
    334338    void membershipEdit(int row) {
    335339        Relation relation = (Relation)membershipData.getValueAt(row, 0);
    … …  
    474478            + tr("Please select the objects you want to change properties for.") + "</p></html>");
    475479
    476     class MemberInfo {
     480    static class MemberInfo {
    477481        List<RelationMember> role = new ArrayList<RelationMember>();
    478482        List<Integer> position = new ArrayList<Integer>();
    … …  
    480484        void add(RelationMember r, Integer p)
    481485        {
    482           role.add(r);
    483           position.add(p);
     486            role.add(r);
     487            position.add(p);
    484488        }
    485489        String getPositionString()
    … …  
    496500                        ++cnt;
    497501                    } else {
    498                         if(cnt == 1)
     502                        if(cnt == 1) {
    499503                            positionString += ","+String.valueOf(last);
    500                         else if(cnt > 1)
     504                        } else if(cnt > 1) {
    501505                            positionString += "-"+String.valueOf(last);
     506                        }
    502507                        positionString += "-"+String.valueOf(cur);
    503508                        cnt = 0;
    … …  
    505510                    last = cur;
    506511                }
    507                 if(cnt == 1)
     512                if(cnt == 1) {
    508513                    positionString += ","+String.valueOf(last);
    509                 else if(cnt > 1)
     514                } else if(cnt > 1) {
    510515                    positionString += "-"+String.valueOf(last);
    511             }
    512             if(positionString.length() > 20)
    513               positionString = positionString.substring(0,17)+"...";
     516                }
     517            }
     518            if(positionString.length() > 20) {
     519                positionString = positionString.substring(0,17)+"...";
     520            }
    514521            return positionString;
    515522        }
    … …  
    589596
    590597        mod.getColumn(1).setCellRenderer(new DefaultTableCellRenderer() {
    591             @SuppressWarnings("unchecked")
    592598            @Override public Component getTableCellRendererComponent(JTable table, Object value,
    593599                    boolean isSelected, boolean hasFocus, int row, int column) {
    … …  
    619625
    620626        mod.getColumn(2).setCellRenderer(new DefaultTableCellRenderer() {
    621             @SuppressWarnings("unchecked")
    622627            @Override public Component getTableCellRendererComponent(JTable table, Object value,
    623628                    boolean isSelected, boolean hasFocus, int row, int column) {
    … …  
    667672
    668673        // -- add action and shortcut
    669         AddAction addAction = new AddAction();
    670674        this.btnAdd = new SideButton(addAction);
    671675        btnAdd.setFocusable(true);
    … …  
    673677        btnAdd.getInputMap(JComponent.WHEN_FOCUSED).put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), "onEnter");
    674678        btnAdd.getActionMap().put("onEnter", addAction);
    675 
    676         Main.registerActionShortcut(addAction, Shortcut.registerShortcut("properties:add", tr("Add Properties"), KeyEvent.VK_B,
    677                 Shortcut.GROUP_MNEMONIC));
    678679
    679680        // -- edit action
    … …  
    873874                    Relation r = (Relation) ref;
    874875                    MemberInfo mi = roles.get(r);
    875                     if(mi == null)
     876                    if(mi == null) {
    876877                        mi = new MemberInfo();
     878                    }
    877879                    roles.put(r, mi);
    878880                    int i = 1;

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTable.java

    r3210 r3327  
    203203            OsmPrimitive primitive = getMemberTableModel().getReferredPrimitive(row);
    204204            layer.data.setSelected(primitive);
    205             AutoScaleAction action = new AutoScaleAction("selection");
    206             action.autoScale();
     205            AutoScaleAction.autoScale("selection");
    207206        }
    208207

  • trunk/src/org/openstreetmap/josm/gui/history/NodeListViewer.java

    r3083 r3327  
    203203            if (p!= null) {
    204204                getEditLayer().data.setSelected(p.getPrimitiveId());
    205                 new AutoScaleAction("selection").autoScale();
     205                AutoScaleAction.autoScale("selection");
    206206            }
    207207        }

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

    r3321 r3327  
    829829        if (data == null)
    830830            return null;
    831         OsmDataLayer layer = Main.main.getEditLayer();
    832831        PresetPanel p = new PresetPanel();
    833832        LinkedList<Item> l = new LinkedList<Item>();
Note: See TracChangeset for help on using the changeset viewer.