Changeset 4596 in josm

Timestamp:

Nov 15, 2011 9:00:55 PM (18 months ago)

Author:

jttt

Message:

Fix #5515 Shortcuts for activating and showing/hiding layers

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java (modified) (6 diffs)

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

@@ -91 +91 @@
 
         MultikeyActionsHandler.getInstance().addAction(instance.new ShowHideLayerAction(false));
+        MultikeyActionsHandler.getInstance().addAction(instance.new ActivateLayerAction());
     }
 
@@ -238 +239 @@
         // -- activate action
         activateLayerAction = new ActivateLayerAction();
+        activateLayerAction.updateEnabledState();
         adaptTo(activateLayerAction, selectionModel);
 
@@ -618 +620 @@
      */
 
-    public final class ActivateLayerAction extends AbstractAction implements IEnabledStateUpdating, MapView.LayerChangeListener{
+    public final class ActivateLayerAction extends AbstractAction implements IEnabledStateUpdating, MapView.LayerChangeListener, MultikeyShortcutAction{
         private  Layer layer;
 
@@ -632 +634 @@
             putValue(SMALL_ICON, ImageProvider.get("dialogs", "activate"));
             putValue(SHORT_DESCRIPTION, tr("Activate the selected layer"));
+            putValue(ACCELERATOR_KEY, Shortcut.registerShortcut("core_multikey:activateLayer", "", 'A', Shortcut.GROUP_DIRECT, KeyEvent.SHIFT_DOWN_MASK + KeyEvent.ALT_DOWN_MASK).getKeyStroke());
             putValue("help", HelpUtil.ht("/Dialog/LayerList#ActivateLayer"));
-            updateEnabledState();
         }
 
@@ -644 +646 @@
                 toActivate = model.getSelectedLayers().get(0);
             }
+            execute(toActivate);
+        }
+
+        private void execute(Layer layer) {
             // model is  going to be updated via LayerChangeListener
             // and PropertyChangeEvents
-            Main.map.mapView.setActiveLayer(toActivate);
-            toActivate.setVisible(true);
+            Main.map.mapView.setActiveLayer(layer);
+            layer.setVisible(true);
         }
 
@@ -681 +687 @@
         public void layerRemoved(Layer oldLayer) {
             updateEnabledState();
+        }
+
+        @Override
+        public void executeMultikeyAction(int index) {
+            Layer l = LayerListDialog.getLayerForIndex(index);
+            if (l != null) {
+                execute(l);
+            }
+        }
+
+        @Override
+        public void repeateLastMultikeyAction() {
+            // Do nothing, repating not supported
+        }
+
+        @Override
+        public List<MultikeyInfo> getMultikeyCombinations() {
+            return LayerListDialog.getLayerInfoByClass(Layer.class);
+        }
+
+        @Override
+        public MultikeyInfo getLastMultikeyAction() {
+            return null; // Repeating action doesn't make much sense for activating
         }
     }