WIP call setLayerElExtent even when features are hidden

Author: AliyanH

src/map-feature.js

@@ -229,14 +229,17 @@ export class MapFeature extends HTMLElement {
           // todo: dynamically update layer bounds of vector layer
           mapmlvectors.layerBounds = M.getBounds(this._layer._content);
           // update map's zoom limit
-          // this._map._addZoomLimit(mapmlvectors);
+          this._map._addZoomLimit(mapmlvectors);
           // TODO: can be set as a handler of featureLayer
           mapmlvectors._resetFeatures();
           L.extend(mapmlvectors.options, mapmlvectors.zoomBounds);
         }
       }
 
-      if (Object.keys(mapmlvectors._layers).length === 1) {
+      // Number of features that are being displayed on the map
+      let renderedFeatureCount = Object.keys(mapmlvectors._layers).length;
+      // 0 because feature could be hidden by the min/max attr., 1 so as other features are added, _setLayerElExtent() is not run multiple times
+      if (renderedFeatureCount === 1 || renderedFeatureCount === 0) {
         this._layer._setLayerElExtent();
       }
       this._setUpEvents();

src/mapml/layers/MapMLLayer.js

@@ -375,11 +375,7 @@ export var MapMLLayer = L.Layer.extend({
               bounds.extend(this[type].layerBounds.max);
             }
           }
-        } else if (
-          // only process extent if mapmlvectors is not empty
-          type === '_mapmlvectors' &&
-          Object.keys(this[type]._layers).length !== 0
-        ) {
+        } else if (type === '_mapmlvectors') {
           if (this[type].layerBounds) {
             if (!bounds) {
               bounds = this[type].layerBounds;