Update how map-feature / mapmlvectors calculates FeatureLayer.layerBounds

Author: prushforth

src/mapml/layers/FeatureLayer.js

@@ -47,13 +47,13 @@ export var FeatureLayer = L.FeatureGroup.extend({
         this._staticFeature = true;
         this.isVisible = true; //placeholder for when this actually gets updated in the future
         this.zoomBounds = M.getZoomBounds(mapml, native.zoom);
-        this.layerBounds = M.getLayerBounds(mapml);
+        this.layerBounds = M.getBounds(mapml);
         L.extend(this.options, this.zoomBounds);
       }
       this.addData(mapml, native.cs, native.zoom);
     } else if (!mapml) {
       this.isVisible = false;
-      this.layerBounds = this.options.extent;
+      this.layerBounds = null;
       this.zoomBounds = this.options.zoomBounds;
     }
   },
@@ -273,6 +273,14 @@ export var FeatureLayer = L.FeatureGroup.extend({
     }
 
     let layer = this.geometryToLayer(mapml, options, nativeCS, +zoom, title);
+    let ext = mapml.extent,
+      xmin = ext.topLeft.pcrs.horizontal,
+      ymin = ext.bottomRight.pcrs.vertical,
+      xmax = ext.bottomRight.pcrs.horizontal,
+      ymax = ext.topLeft.pcrs.vertical,
+      bnd = L.bounds(L.point(xmin, ymin), L.point(xmax, ymax));
+
+    this.layerBounds = this.layerBounds ? this.layerBounds.extend(bnd) : bnd;
     if (layer) {
       // if the layer is being used as a query handler output, it will have
       // a color option set.  Otherwise, copy classes from the feature

src/mapml/layers/MapMLLayer.js

@@ -1515,7 +1515,6 @@ export var MapMLLayer = L.Layer.extend({
           pane: layer._container,
           opacity: layer.options.opacity,
           projection: layer._properties.projection,
-          extent: M.getBounds(layer._content),
           native: native,
           zoomBounds: M.getZoomBounds(layer._content, native.zoom),
           // each owned child layer gets a reference to the root layer