@defra/interactive-map 0.0.36-alpha → 0.0.37-alpha

package/plugins/datasets/src/adapters/esri/esriLayerAdapter.js

@@ -49,9 +49,21 @@ export default class EsriLayerAdapter extends LayerAdapter {
 
     // Finally show all layers that are visible based on the dataset/mapStyle visibility
     await Promise.all(topLevelDatasets.map(registryDataset => this.applyDatasetVisibility(registryDataset.id)))
+    this._reorderLayers()
     this.resolveReady()
   }
 
+  _reorderLayers () {
+    // Ensure that all sketch layers are on top of the map, so that they are not obscured by other layers
+    // Only applicable when the draw plugin is in use, but safe to call regardless
+    const layersLength = this._map?.allLayers?.items?.length
+    if (layersLength) {
+      this._map.allLayers.items
+        .filter((layer) => layer.id.includes('ketchLayer'))
+        .forEach((layer) => this._map.reorder(layer, layersLength))
+    }
+  }
+
   _addGroupLayer (esriGroupId) {
     // Either adds a new group layer to the map, or returns an existing one if it already exists
     if (!this._groupLayers[esriGroupId]) {
@@ -117,6 +129,7 @@ export default class EsriLayerAdapter extends LayerAdapter {
     await this._addLayers(registryDataset)
     const { parentId } = registryDataset
     const vectorTileLayer = this._mapVisibilityLayers[parentId || datasetId]
+    this._reorderLayers()
     this.applyDatasetOpacity(datasetId)
     this._applyStyleLayerPaintProperties(registryDataset, vectorTileLayer)
     this.applyDatasetVisibility(datasetId)