chore: do not try to remove an icon not in DOM

This occurs when using cluster, which already add/remove markers
umap-project · Oct 10, 2024 · ca3451c · ca3451c
1 parent c6da1f9
commit ca3451c
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/umap/static/umap/js/modules/rendering/ui.js b/umap/static/umap/js/modules/rendering/ui.js
@@ -36,6 +36,12 @@ const FeatureMixin = {
     }
   },
 
+  _removeIcon: function () {
+    // It may not be in the DOM, and Leaflet does not deal with this
+    // situation
+    if (this._icon) Marker.prototype._removeIcon.call(this)
+  },
+
   addInteractions: function () {
     this.on('contextmenu editable:vertex:contextmenu', this.onContextMenu)
     this.on('click', this.onClick)