Changeset 1945 in josm

Timestamp:

Aug 10, 2009 7:15:10 AM (4 years ago)

Author:

Gubaer

Message:

fixed #3211: Keystrokes / "+" no function / "Del" removes selected object instead of selected list item

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/PropertiesDialog.java (modified) (8 diffs)

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

@@ -33 +33 @@
 import java.util.Map.Entry;
 
+import javax.swing.AbstractAction;
 import javax.swing.Box;
 import javax.swing.DefaultListCellRenderer;
 import javax.swing.JComboBox;
+import javax.swing.JComponent;
 import javax.swing.JDialog;
 import javax.swing.JLabel;
@@ -43 +45 @@
 import javax.swing.JScrollPane;
 import javax.swing.JTable;
+import javax.swing.KeyStroke;
 import javax.swing.ListSelectionModel;
+import javax.swing.event.ListSelectionEvent;
+import javax.swing.event.ListSelectionListener;
 import javax.swing.table.DefaultTableCellRenderer;
 import javax.swing.table.DefaultTableModel;
@@ -72 +77 @@
 import org.openstreetmap.josm.tools.AutoCompleteComboBox;
 import org.openstreetmap.josm.tools.GBC;
+import org.openstreetmap.josm.tools.ImageProvider;
 import org.openstreetmap.josm.tools.Shortcut;
 
@@ -413 +419 @@
 
     /**
-     * Delete the keys from the given row.
-     * @param row   The row, which key gets deleted from the dataset.
-     */
-    private void delete(int row) {
-        String key = propertyData.getValueAt(row, 0).toString();
-        Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
-        Main.main.undoRedo.add(new ChangePropertyCommand(sel, key, null));
-        DataSet.fireSelectionChanged(sel);
-        selectionChanged(sel); // update table
-
-        int rowCount = propertyTable.getRowCount();
-        propertyTable.changeSelection((row < rowCount ? row : (rowCount-1)), 0, false, false);
-    }
-
-    /**
      * The property data.
      */
@@ -579 +570 @@
                     if (e.getActionCommand().equals("Edit")) {
                         membershipEdit(row);
-                    } else if (e.getActionCommand().equals("Delete")) {
-                        Relation cur = (Relation)membershipData.getValueAt(row, 0);
-                        int result = new ExtendedDialog(Main.parent,
-                                tr("Change relation"),
-                                tr("Really delete selection from relation {0}?", NAME_FORMATTER.getName(cur)),
-                                new String[] {tr("Delete from relation"), tr("Cancel")},
-                                new String[] {"dialogs/delete.png", "cancel.png"}).getValue();
-
-                        if(result == 1)
-                        {
-                            Relation rel = new Relation(cur);
-                            Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
-                            for (RelationMember rm : cur.getMembers()) {
-                                for (OsmPrimitive osm : sel) {
-                                    if (rm.getMember() == osm)
-                                    {
-                                        rel.members.remove(rm);
-                                        break;
-                                    }
-                                }
-                            }
-                            Main.main.undoRedo.add(new ChangeCommand(cur, rel));
-                            DataSet.fireSelectionChanged(sel);
-                            selectionChanged(sel); // update whole table
-                        }
-
                     }
                 }
@@ -614 +579 @@
                     if (e.getActionCommand().equals("Edit")) {
                         propertyEdit(sel >= 0 ? sel : 0);
-                    } else if (e.getActionCommand().equals("Delete")) {
-                        delete(sel >= 0 ? sel : 0);
                     }
                 }
@@ -633 +596 @@
         buttonPanel.add(this.btnEdit);
 
-        s = Shortcut.registerShortcut("properties:delete", tr("Delete Properties"), KeyEvent.VK_Q,
-                Shortcut.GROUP_MNEMONIC);
-        this.btnDel = new SideButton(marktr("Delete"),"delete","Properties",
-                tr("Delete the selected key in all objects"), s, buttonAction);
+        // -- delete action
+        DeleteAction deleteAction = new DeleteAction();
+        this.btnDel = new SideButton(deleteAction);
+        membershipTable.getSelectionModel().addListSelectionListener(deleteAction);
+        propertyTable.getSelectionModel().addListSelectionListener(deleteAction);
+        getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT).put(
+                KeyStroke.getKeyStroke(KeyEvent.VK_DELETE, 0),"delete"
+        );
+        getActionMap().put("delete", deleteAction);
         buttonPanel.add(this.btnDel);
         add(buttonPanel, BorderLayout.SOUTH);
@@ -860 +828 @@
 
 
+    class DeleteAction extends AbstractAction implements ListSelectionListener {
+
+        protected void deleteProperty(int row){
+            String key = propertyData.getValueAt(row, 0).toString();
+            Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
+            Main.main.undoRedo.add(new ChangePropertyCommand(sel, key, null));
+            DataSet.fireSelectionChanged(sel);
+            selectionChanged(sel); // update table
+
+            int rowCount = propertyTable.getRowCount();
+            propertyTable.changeSelection((row < rowCount ? row : (rowCount-1)), 0, false, false);
+        }
+
+        protected void deleteFromRelation(int row) {
+            Relation cur = (Relation)membershipData.getValueAt(row, 0);
+            int result = new ExtendedDialog(Main.parent,
+                    tr("Change relation"),
+                    tr("Really delete selection from relation {0}?", NAME_FORMATTER.getName(cur)),
+                    new String[] {tr("Delete from relation"), tr("Cancel")},
+                    new String[] {"dialogs/delete.png", "cancel.png"}).getValue();
+
+            if(result != 1)
+                return;
+
+            Relation rel = new Relation(cur);
+            Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
+            for (RelationMember rm : cur.getMembers()) {
+                for (OsmPrimitive osm : sel) {
+                    if (rm.getMember() == osm)
+                    {
+                        rel.members.remove(rm);
+                        break;
+                    }
+                }
+            }
+            Main.main.undoRedo.add(new ChangeCommand(cur, rel));
+            DataSet.fireSelectionChanged(sel);
+            selectionChanged(sel); // update whole table
+        }
+
+        public DeleteAction() {
+            putValue(NAME, tr("Delete"));
+            putValue(SHORT_DESCRIPTION, tr("Delete the selected key in all objects"));
+            putValue(SMALL_ICON, ImageProvider.get("dialogs", "delete"));
+            Shortcut s = Shortcut.registerShortcut("properties:delete", tr("Delete Properties"), KeyEvent.VK_Q,
+                    Shortcut.GROUP_MNEMONIC);
+            putValue(MNEMONIC_KEY, (int) KeyEvent.getKeyText(s.getAssignedKey()).charAt(0));
+            updateEnabledState();
+        }
+
+        public void actionPerformed(ActionEvent e) {
+            if (propertyTable.getSelectedRowCount() >0 ) {
+                int row = propertyTable.getSelectedRow();
+                deleteProperty(row);
+            } else if (membershipTable.getSelectedRowCount() > 0) {
+                int row = membershipTable.getSelectedRow();
+                deleteFromRelation(row);
+            }
+        }
+
+        protected void updateEnabledState() {
+            setEnabled(
+                    PropertiesDialog.this.propertyTable.getSelectedRowCount() >0
+                    || PropertiesDialog.this.membershipTable.getSelectedRowCount() > 0
+            );
+        }
+
+        public void valueChanged(ListSelectionEvent e) {
+            updateEnabledState();
+        }
+    }
+
 }