Changeset 2759 in josm

Timestamp:

Jan 7, 2010 7:29:53 PM (3 years ago)

Author:

mjulius

Message:

Make the new zoom previous and next actions listen to zoom changes and set enabled state.

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/AutoScaleAction.java (modified) (3 diffs)
• actions/JosmAction.java (modified) (1 diff)
• gui/NavigatableComponent.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/AutoScaleAction.java

@@ -18 +18 @@
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
+import org.openstreetmap.josm.gui.MapView;
 import org.openstreetmap.josm.gui.dialogs.LayerListDialog;
 import org.openstreetmap.josm.gui.layer.Layer;
@@ -226 +227 @@
                 setEnabled(true);
             }
+        } else if ("previous".equals(mode)) {
+            setEnabled(Main.map != null && Main.map.mapView != null && Main.map.mapView.hasZoomUndoEntries());
+        } else if ("next".equals(mode)) {
+            setEnabled(Main.map != null && Main.map.mapView != null && Main.map.mapView.hasZoomRedoEntries());
         } else {
             setEnabled(
@@ -240 +245 @@
         }
     }
+
+    @Override
+    protected void installAdapters() {
+        super.installAdapters();
+        // make this action listen to zoom change events
+        //
+        zoomChangeAdapter = new ZoomChangeAdapter();
+        MapView.addZoomChangeListener(zoomChangeAdapter);
+        initEnabledState();
+    }
+
+    /**
+     * Adapter for selection change events
+     *
+     */
+    private class ZoomChangeAdapter implements MapView.ZoomChangeListener {
+        public void zoomChanged() {
+            updateEnabledState();
+        }
+    }
+
+    private ZoomChangeAdapter zoomChangeAdapter;
 }

trunk/src/org/openstreetmap/josm/actions/JosmAction.java

@@ -139 +139 @@
     }
 
-    private void installAdapters() {
+    protected void installAdapters() {
         // make this action listen to layer change and selection change events
         //

trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java

@@ -13 +13 @@
 import java.util.List;
 import java.util.TreeMap;
+import java.util.concurrent.CopyOnWriteArrayList;
 
 import javax.swing.JComponent;
@@ -38 +39 @@
  */
 public class NavigatableComponent extends JComponent implements Helpful {
+
+    /**
+     * Interface to notify listeners of the change of the zoom area.
+     */
+    public interface ZoomChangeListener {
+        void zoomChanged();
+    }
+
+    /**
+     * the zoom listeners
+     */
+    private static final CopyOnWriteArrayList<ZoomChangeListener> zoomChangeListeners = new CopyOnWriteArrayList<ZoomChangeListener>();
+
+    /**
+     * Removes a zoom change listener
+     *
+     * @param listener the listener. Ignored if null or already absent
+     */
+    public static void removeZoomChangeListener(NavigatableComponent.ZoomChangeListener listener) {
+        zoomChangeListeners.remove(listener);
+    }
+
+    /**
+     * Adds a zoom change listener
+     *
+     * @param listener the listener. Ignored if null or already registered.
+     */
+    public static void addZoomChangeListener(NavigatableComponent.ZoomChangeListener listener) {
+        if (listener != null) {
+            zoomChangeListeners.addIfAbsent(listener);
+        }
+    }
+
+    protected static void fireZoomChanged() {
+        for (ZoomChangeListener l : zoomChangeListeners) {
+            l.zoomChanged();
+        }
+    }
+
 
     public static final int snapDistance = Main.pref.getInteger("node.snap-distance", 10);
@@ -267 +307 @@
 
         repaint();
+        fireZoomChanged();
     }
 
@@ -367 +408 @@
             zoomNoUndoTo(zoom.getCenterEastNorth(), zoom.getScale());
         }
+    }
+
+    public boolean hasZoomUndoEntries() {
+        return !zoomUndoBuffer.isEmpty();
+    }
+
+    public boolean hasZoomRedoEntries() {
+        return !zoomRedoBuffer.isEmpty();
     }