From 82708b1e53562ca55f6b2734d8dc07df951e9728 Mon Sep 17 00:00:00 2001
From: Yohan Boniface <yohanboniface@free.fr>
Date: Tue, 17 Dec 2024 15:29:55 +0100
Subject: [PATCH] fix: bring marker to front on highlight

fix #2351
---
 umap/static/umap/js/modules/rendering/ui.js | 2 ++
 umap/static/umap/map.css                    | 1 -
 umap/static/umap/vars.css                   | 1 -
 3 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/umap/static/umap/js/modules/rendering/ui.js b/umap/static/umap/js/modules/rendering/ui.js
index f2c307ee..67add220 100644
--- a/umap/static/umap/js/modules/rendering/ui.js
+++ b/umap/static/umap/js/modules/rendering/ui.js
@@ -239,10 +239,12 @@ export const LeafletMarker = Marker.extend({
 
   highlight: function () {
     DomUtil.addClass(this.options.icon.elements.main, 'umap-icon-active')
+    this._bringToFront()
   },
 
   resetHighlight: function () {
     DomUtil.removeClass(this.options.icon.elements.main, 'umap-icon-active')
+    this._resetZIndex()
   },
 
   getPopupToolbarAnchor: function () {
diff --git a/umap/static/umap/map.css b/umap/static/umap/map.css
index ea74ecef..156dd791 100644
--- a/umap/static/umap/map.css
+++ b/umap/static/umap/map.css
@@ -924,7 +924,6 @@ a.umap-control-caption,
     width: 2px;
 }
 .umap-icon-active {
-    z-index: var(--zindex-icon-active)!important;
     opacity: 1.0!important;
 }
 .umap-edit-enabled .readonly {
diff --git a/umap/static/umap/vars.css b/umap/static/umap/vars.css
index 1493757b..4d755937 100644
--- a/umap/static/umap/vars.css
+++ b/umap/static/umap/vars.css
@@ -48,7 +48,6 @@
     --zindex-autocomplete: 470;
     --zindex-dialog: 460;
     --zindex-contextmenu: 455;
-    --zindex-icon-active: 450;
     --zindex-tooltip: 445;
     --zindex-panels: 440;
     --zindex-controls: 430;