Bug/minor fixes

demo/js/index.js

@@ -22,8 +22,10 @@ var interactPlugin = createInteractPlugin({
 	},{
 		layerId: 'linked-parcels',
 		// idProperty: 'id'
+	},{
+		layerId: 'OS/TopographicArea_1/Agricultural Land'
 	}],
-	interactionMode: 'auto', // 'auto', 'select', 'marker' // defaults to 'marker'
+	interactionMode: 'select', // 'auto', 'select', 'marker' // defaults to 'marker'
 	multiSelect: true,
 	contiguous: true,
 	// excludeModes: ['draw']
@@ -106,6 +108,7 @@ var interactiveMap = new InteractiveMap('map', {
 	containerHeight: '650px',
 	transformRequest: transformTileRequest,
 	enableZoomControls: true,
+	readMapText: true,
 	// enableFullscreen: true,
 	// hasExitButton: true,
 	// markers: [{
@@ -149,7 +152,6 @@ interactiveMap.on('app:ready', function (e) {
 })
 
 interactiveMap.on('map:ready', function (e) {
-	// console.log('map:ready')
 	// framePlugin.addFrame('test', {
 	// 	aspectRatio: 1
 	// })
@@ -178,7 +180,7 @@ interactiveMap.on('draw:ready', function () {
 	// drawPlugin.split('test1234', {
 	// 	snapLayers: ['OS/TopographicArea_1/Agricultural Land']
 	// })
-	// drawPlugin.newLine('test', {
+	// drawPlugin.newPolygon('test', {
 	// 	snapLayers: ['OS/TopographicArea_1/Agricultural Land']
 	// })
 	// drawPlugin.editFeature('test1234')

demo/js/planning.js

@@ -131,7 +131,7 @@ const interactiveMap = new InteractiveMap('map', {
 	// search
 })
 
-interactiveMap.on('map:ready', function (e) {
+interactiveMap.on('app:ready', function (e) {
 	interactiveMap.addButton('menu', {
 		label: 'Menu',
 		panelId: 'menu',

docs/api.md

@@ -388,6 +388,14 @@ See [PluginDescriptor](./plugins/plugin-descriptor.md) for full details.
 
 ---
 
+### `preserveStateOnClose`
+**Type:** `boolean`  
+**Default:** `false`  
+
+Controls whether closing the map (via the browser back button or the exit map button when `hasExitButton` is `true` and the map is fullscreen) destroys the map instance or hides it while preserving its current state. Set to `true` to keep the map state intact, which is useful for implementations like a toggle map view list view pattern.
+
+---
+
 ### `readMapText`
 **Type:** `boolean`
 **Default:** `false`

plugins/beta/draw-ml/src/mapboxSnap.js

@@ -140,6 +140,8 @@ function applyMapboxSnapPatches(colors) {
 function pollUntil(checkFn, onSuccess) {
   (function poll() {
     const result = checkFn()
+    // null signals to stop polling, falsy continues polling
+    if (result === null) return
     result ? onSuccess(result) : requestAnimationFrame(poll)
   })()
 }
@@ -267,7 +269,7 @@ export function initMapLibreSnap(map, draw, snapOptions = {}) {
   // Handle style changes - re-patch source and ensure snap layer exists
   map.on('style.load', () => {
     pollUntil(
-      () => map.getSource('mapbox-gl-draw-hot'),
+      () => map._removed ? null : map.getSource('mapbox-gl-draw-hot'),
       (source) => {
         patchSourceData(source)
 
@@ -312,7 +314,7 @@ export function initMapLibreSnap(map, draw, snapOptions = {}) {
 
   // Initial setup - poll until draw source exists
   pollUntil(
-    () => map.getSource('mapbox-gl-draw-hot'),
+    () => map._removed ? null : map.getSource('mapbox-gl-draw-hot'),
     createSnap
   )
 }

plugins/interact/src/hooks/useInteractionHandlers.js

@@ -62,7 +62,14 @@ export const useInteractionHandlers = ({
   useEffect(() => {
     // Skip if features exist but bounds not yet calculated
     const awaitingBounds = selectedFeatures.length > 0 && !selectionBounds
-    if (awaitingBounds || selectedFeatures === lastEmittedSelectionChange.current) {
+    if (awaitingBounds) {
+      return
+    }
+
+    // Skip if selection was already empty and remains empty
+    const prev = lastEmittedSelectionChange.current
+    const wasEmpty = prev === null || prev.length === 0
+    if (wasEmpty && selectedFeatures.length === 0) {
       return
     }