From 3115038ba6c875a5184fcabf6bf443bdb075487d Mon Sep 17 00:00:00 2001 From: Yohan Boniface Date: Mon, 31 Jul 2023 20:59:09 +0200 Subject: [PATCH] Fix popup panel not opening from the data browser The issue was: - clicking on feature in the data browser replaces the panel content - then the clicked link (the feature name or the little glass icon) is removed - thus it is detached from the DOM - and specially from its parent on which disableClickPropagation was called - so Leaflet fails to prevent click propagation - so the map received the click, and thus it call the onRemove method on the panel, as expected (clicking on the map always close the current open popup, if any) This could be removed when this is released: https://github.com/Leaflet/Leaflet/pull/9052 --- umap/static/umap/js/umap.controls.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/umap/static/umap/js/umap.controls.js b/umap/static/umap/js/umap.controls.js index 0c1bc9c1..06fd9e14 100644 --- a/umap/static/umap/js/umap.controls.js +++ b/umap/static/umap/js/umap.controls.js @@ -671,6 +671,9 @@ L.U.DataLayer.addInitHook(function () { L.U.Map.include({ _openBrowser: function () { const browserContainer = L.DomUtil.create('div', 'umap-browse-data') + // HOTFIX. Remove when this is merged and released: + // https://github.com/Leaflet/Leaflet/pull/9052 + L.DomEvent.disableClickPropagation(browserContainer) const title = L.DomUtil.add( 'h3',