From 39333953d70e4112ba138c4e37ce4024d7eb3421 Mon Sep 17 00:00:00 2001
From: orangemug <info@orangemug.co.uk>
Date: Mon, 17 Feb 2020 13:24:14 +0000
Subject: [PATCH] Only show 'switch to layer' button if layer not already
 selected.

---
 src/components/App.jsx          |  1 +
 src/components/MessagePanel.jsx | 22 ++++++++++++++--------
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/components/App.jsx b/src/components/App.jsx
index 3e0158c8..6273df33 100644
--- a/src/components/App.jsx
+++ b/src/components/App.jsx
@@ -732,6 +732,7 @@ export default class App extends React.Component {
     /> : null
 
     const bottomPanel = (this.state.errors.length + this.state.infos.length) > 0 ? <MessagePanel
+      currentLayer={selectedLayer}
       onLayerSelect={this.onLayerSelect}
       mapStyle={this.state.mapStyle}
       errors={this.state.errors}
diff --git a/src/components/MessagePanel.jsx b/src/components/MessagePanel.jsx
index 373691eb..f114a8c4 100644
--- a/src/components/MessagePanel.jsx
+++ b/src/components/MessagePanel.jsx
@@ -7,6 +7,7 @@ class MessagePanel extends React.Component {
     infos: PropTypes.array,
     mapStyle: PropTypes.object,
     onLayerSelect: PropTypes.func,
+    currentLayer: PropTypes.object,
   }
 
   static defaultProps = {
@@ -18,17 +19,22 @@ class MessagePanel extends React.Component {
       let content;
       if (error.parsed && error.parsed.type === "layer") {
         const {parsed} = error;
-        const {mapStyle} = this.props;
+        const {mapStyle, currentLayer} = this.props;
         const layerId = mapStyle.layers[parsed.data.index].id;
         content = (
           <>
-            Layer <span>&apos;{layerId}&apos;</span>: {parsed.data.message} &mdash;&nbsp;
-            <button
-              className="maputnik-message-panel__switch-button"
-              onClick={() => this.props.onLayerSelect(layerId)}
-            >
-              switch to layer
-            </button>
+            Layer <span>&apos;{layerId}&apos;</span>: {parsed.data.message}
+            {currentLayer.id !== layerId &&
+              <>
+                &nbsp;&mdash;&nbsp;
+                <button
+                  className="maputnik-message-panel__switch-button"
+                  onClick={() => this.props.onLayerSelect(layerId)}
+                >
+                  switch to layer
+                </button>
+              </>
+            }
           </>
         );
       }