From: Tom Hughes <tom@compton.nu>
Date: Tue, 22 Feb 2022 18:54:40 +0000 (+0000)
Subject: Don't highlight selected items in the data layer
X-Git-Tag: live~2326
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/97bb13595750d2ba5d03a5abe2ede7cd11a8a481?hp=b66f7a30d035c92708f309ed11f589f0d88bd03a

Don't highlight selected items in the data layer

Routing to the feature page is going to cause them to be overlaid
in orange anyway so there's no need for the blue highlight.
---

diff --git a/app/assets/javascripts/index/browse.js b/app/assets/javascripts/index/browse.js
index 779a21d69..3f888c1cd 100644
--- a/app/assets/javascripts/index/browse.js
+++ b/app/assets/javascripts/index/browse.js
@@ -1,6 +1,5 @@
 OSM.initializeBrowse = function (map) {
   var browseBounds;
-  var selectedLayer;
   var dataLayer = map.dataLayer;
 
   dataLayer.setStyle({
@@ -92,7 +91,6 @@ OSM.initializeBrowse = function (map) {
       url: url,
       success: function (xml) {
         dataLayer.clearLayers();
-        selectedLayer = null;
 
         var features = dataLayer.buildFeatures(xml);
 
@@ -118,18 +116,6 @@ OSM.initializeBrowse = function (map) {
   }
 
   function onSelect(layer) {
-    // Unselect previously selected feature
-    if (selectedLayer) {
-      selectedLayer.setStyle(selectedLayer.originalStyle);
-    }
-
-    // Redraw in selected style
-    layer.originalStyle = { color: layer.options.color, weight: layer.options.weight };
-    layer.setStyle({ color: "#0000ff", weight: 8 });
-
     OSM.router.route("/" + layer.feature.type + "/" + layer.feature.id);
-
-    // Stash the currently drawn feature
-    selectedLayer = layer;
   }
 };