Ticket #5781: arrow-keys-property-table-proof-of-concept.patch

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

@@ -31 +31 @@
 import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.List;
-import java.util.Map.Entry;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Set;
 import java.util.TreeMap;
 import java.util.TreeSet;
@@ -65 +65 @@
 
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.JosmAction;
+import org.openstreetmap.josm.actions.search.SearchAction.SearchMode;
+import org.openstreetmap.josm.actions.search.SearchAction.SearchSetting;
 import org.openstreetmap.josm.command.ChangeCommand;
 import org.openstreetmap.josm.command.ChangePropertyCommand;
 import org.openstreetmap.josm.command.Command;
@@ -102 +104 @@
 import org.openstreetmap.josm.tools.OpenBrowser;
 import org.openstreetmap.josm.tools.Shortcut;
 import org.openstreetmap.josm.tools.Utils;
-import org.openstreetmap.josm.actions.search.SearchAction.SearchMode;
-import org.openstreetmap.josm.actions.search.SearchAction.SearchSetting;
 
 /**
  * This dialog displays the properties of the current selected primitives.
@@ -378 +378 @@
                 }
         }
         propertyTable.changeSelection(row, 0, false, false);
+        propertyTable.requestFocusInWindow();
     }
 
     /**
@@ -487 +488 @@
         lastAddKey = key;
         lastAddValue = value;
         Main.main.undoRedo.add(new ChangePropertyCommand(sel, key, value));
-        btnAdd.requestFocusInWindow();
+        propertyTable.requestFocusInWindow(); // necessary?
     }
 
     /**
@@ -788 +789 @@
 
         // -- add action and shortcut
         this.btnAdd = new SideButton(addAction);
-        btnAdd.setFocusable(true);
-        btnAdd.getInputMap(JComponent.WHEN_FOCUSED).put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), "onEnter");
-        btnAdd.getActionMap().put("onEnter", addAction);
+        getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT).put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), "onEnter");
+        getActionMap().put("onEnter", addAction);
+        // disable the default enter keybinding in the tables:
+        propertyTable.getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT).put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), "none");
+        
 
         // -- edit action
         //