fix(layer-selector): do not set opacity if not using slider

Co-authored-by: Scott Davis <[email protected]>
agrc · Sep 6, 2022 · f160305 · f160305
1 parent 81e3035
commit f160305
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/packages/layer-selector/src/LayerSelector.jsx b/packages/layer-selector/src/LayerSelector.jsx
@@ -184,13 +184,15 @@ const LayerSelector = (props) => {
 
   const [opacity, setOpacity] = useState(1);
   useEffect(() => {
-    for (const layerId in managedLayers) {
-      const managedLayer = managedLayers[layerId];
-      if (managedLayer.layer) {
-        managedLayer.layer.opacity = opacity;
+    if (props.showOpacitySlider) {
+      for (const layerId in managedLayers) {
+        const managedLayer = managedLayers[layerId];
+        if (managedLayer.layer) {
+          managedLayer.layer.opacity = opacity;
+        }
       }
     }
-  }, [managedLayers, opacity]);
+  }, [props.showOpacitySlider, managedLayers, opacity]);
 
   useEffect(() => {
     const managedLayersDraft = { ...managedLayers };