Changeset 36125 in osm for applications/editors/josm/plugins

Timestamp:

2023-08-28T22:14:59+02:00 (15 months ago)

Author:

taylor.smock

Message:

Improve Zoom to layer action for PMTiles layers

Location:

applications/editors/josm/plugins/pmtiles/src/main/java/org/openstreetmap/josm/plugins/pmtiles/gui/layers

Files:

• PMTilesImageLayer.java (modified) (2 diffs)
• PMTilesLayer.java (modified) (2 diffs)
• PMTilesMVTLayer.java (modified) (2 diffs)

applications/editors/josm/plugins/pmtiles/src/main/java/org/openstreetmap/josm/plugins/pmtiles/gui/layers/PMTilesImageLayer.java

@@ -7 +7 @@
 import org.openstreetmap.gui.jmapviewer.interfaces.TileLoader;
 import org.openstreetmap.josm.data.imagery.vectortile.mapbox.MVTFile;
+import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
 import org.openstreetmap.josm.gui.layer.AbstractCachedTileSourceLayer;
 import org.openstreetmap.josm.plugins.pmtiles.data.imagery.PMTilesImageryInfo;
@@ -69 +70 @@
         return PMTilesLayer.super.getChangesetSourceTag();
     }
+
+    @Override
+    public void visitBoundingBox(BoundingXYVisitor v) {
+        super.visitBoundingBox(v);
+        PMTilesLayer.super.visitBoundingBox(v);
+    }
 }

applications/editors/josm/plugins/pmtiles/src/main/java/org/openstreetmap/josm/plugins/pmtiles/gui/layers/PMTilesLayer.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.Utils.getSystemProperty;
 
+import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
 import org.openstreetmap.josm.plugins.pmtiles.data.imagery.PMTilesImageryInfo;
 import org.openstreetmap.josm.tools.TextUtils;
@@ -49 +50 @@
         return sb.toString();
     }
+
+    /**
+     * Visits the content bounds of this layer. The behavior of this method depends on the layer,
+     * but each implementation should attempt to cover the relevant content of the layer in this method.
+     * @param v The visitor that gets notified about the contents of this layer.
+     * @see org.openstreetmap.josm.gui.layer.Layer#visitBoundingBox
+     */
+    default void visitBoundingBox(BoundingXYVisitor v) {
+        v.visit(this.getInfo().getBounds());
+    }
 }

applications/editors/josm/plugins/pmtiles/src/main/java/org/openstreetmap/josm/plugins/pmtiles/gui/layers/PMTilesMVTLayer.java

@@ -8 +8 @@
 import org.openstreetmap.josm.data.imagery.vectortile.mapbox.MVTFile;
 import org.openstreetmap.josm.data.imagery.vectortile.mapbox.MapboxVectorTileSource;
+import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
 import org.openstreetmap.josm.gui.layer.imagery.MVTLayer;
 import org.openstreetmap.josm.plugins.pmtiles.data.imagery.PMTilesImageryInfo;
@@ -70 +71 @@
         return PMTilesLayer.super.getChangesetSourceTag();
     }
+
+    @Override
+    public void visitBoundingBox(BoundingXYVisitor v) {
+        super.visitBoundingBox(v);
+        PMTilesLayer.super.visitBoundingBox(v);
+    }
 }