mobility-toolbox-js 3.0.0-beta.3 → 3.0.0-beta.5

package/ol/layers/MaplibreStyleLayer.js

@@ -135,25 +135,27 @@ class MaplibreStyleLayer extends MobilityLayerMixin(Layer) {
         if (!this.map || !this.maplibreLayer) {
             return;
         }
-        // Apply the initial visibiltity.
-        const { maplibreMap } = this.maplibreLayer;
-        if (!maplibreMap) {
-            // If the maplibreMap is not yet created because the  map has no target yet, we
+        if (!this.map.getTargetElement()) {
+            // If ther e is no target element the maplibreMap is not yet created, we
             // relaunch the initialisation when it's the case.
             this.olListenersKeys.push(this.map.on('change:target', () => {
                 this.attachToMap(map);
             }));
             return;
         }
-        // maplibreMap.loaded() and maplibreMap.isStyleLoaded() are reliable only on the first call of init.
-        // On the next call (when a topic change for example), these functions returns false because
-        // the style is being modified.
-        // That's why we rely on a property instead for the next calls.
-        if (maplibreMap.loaded()) {
-            this.onLoad();
-        }
-        else {
-            maplibreMap.once('load', this.onLoad);
+        // Apply the initial visibility if possible otherwise we wait for the load event of the layer
+        const { maplibreMap } = this.maplibreLayer;
+        if (maplibreMap) {
+            // maplibreMap.loaded() and maplibreMap.isStyleLoaded() are reliable only on the first call of init.
+            // On the next call (when a topic change for example), these functions returns false because
+            // the style is being modified.
+            // That's why we rely on a property instead for the next calls.
+            if (maplibreMap.loaded()) {
+                this.onLoad();
+            }
+            else {
+                maplibreMap.once('load', this.onLoad);
+            }
         }
         // Apply the visibiltity when layer's visibility change.
         this.olListenersKeys.push(

package/ol/mixins/PropertiesLayerMixin.js

@@ -189,7 +189,7 @@ function PropertiesLayerMixin(Base) {
                 this.map.removeLayer(child);
             });
             // @ts-ignore
-            (super.detachFromMap || (() => { }))(map);
+            (super.detachFromMap || (() => { }))();
         }
         /**
          * Return the an array containing all the descendants of the layer in a flat array. Including the current layer.

package/package.json

@@ -2,7 +2,7 @@
   "name": "mobility-toolbox-js",
   "license": "MIT",
   "description": "Toolbox for JavaScript applications in the domains of mobility and logistics.",
-  "version": "3.0.0-beta.3",
+  "version": "3.0.0-beta.5",
   "homepage": "https://mobility-toolbox-js.geops.io/",
   "module": "index.js",
   "exports": {