@eeacms/volto-arcgis-block 0.1.139 → 0.1.141

package/CHANGELOG.md

@@ -4,7 +4,16 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
-### [0.1.139](https://github.com/eea/volto-arcgis-block/compare/0.1.138...0.1.139) - 9 May 2023
+### [0.1.141](https://github.com/eea/volto-arcgis-block/compare/0.1.140...0.1.141) - 11 May 2023
+
+### [0.1.140](https://github.com/eea/volto-arcgis-block/compare/0.1.139...0.1.140) - 11 May 2023
+
+#### :hammer_and_wrench: Others
+
+- CLMS-1953 (lint): Sets only one span with text. Can handle layers with no legends [ujbolivar - [`4335250`](https://github.com/eea/volto-arcgis-block/commit/433525063af14ac1e48d9a5bb646127769efb5e2)]
+- CLMS-1953 (lint): Ran linting scripts [ujbolivar - [`b73661d`](https://github.com/eea/volto-arcgis-block/commit/b73661d000cda6629be6dd0b110d5acb96c1e1bf)]
+- CLMS-1953 (chore): cleaned up and took out unecessary functions [ujbolivar - [`60cd5f8`](https://github.com/eea/volto-arcgis-block/commit/60cd5f87f7737bfda0ae801b3acf7627bfe2c182)]
+### [0.1.139](https://github.com/eea/volto-arcgis-block/compare/0.1.138...0.1.139) - 10 May 2023
 
 #### :hammer_and_wrench: Others
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eeacms/volto-arcgis-block",
-  "version": "0.1.139",
+  "version": "0.1.141",
   "description": "volto-arcgis-block: Volto add-on",
   "main": "src/index.js",
   "author": "European Environment Agency: CodeSyntax",

package/src/components/MapViewer/LegendWidget.jsx

@@ -34,11 +34,15 @@ class LegendWidget extends React.Component {
 
       // If img src returns a broken link
       if (!(img.complete && img.naturalHeight !== 0)) {
-        img.style.display = 'none';
+        if (img.style) {
+          img.style.display = 'none';
 
-        let span = document.createElement('span');
-        span.innerHTML = 'No legend available';
-        element.parentNode.appendChild(span);
+          if (element.parentNode.querySelector('span')) return;
+
+          let span = document.createElement('span');
+          span.innerHTML = 'No legend available';
+          element.parentNode.appendChild(span);
+        }
       }
     });
   }

package/src/components/MapViewer/MenuWidget.jsx

@@ -648,25 +648,6 @@ class MenuWidget extends React.Component {
     }, 1000);
   }
 
-  setLegendOpacity = () => {
-    const collection = document.getElementsByClassName('esri-legend__symbol');
-
-    Array.prototype.forEach.call(collection, function (element) {
-      let img = {};
-
-      if (element.hasChildNodes()) {
-        img = element.childNodes[0];
-      } else {
-        img = element;
-      }
-      // Set Legend opacity back to 1;
-      img.setAttribute(
-        'style',
-        'opacity:1; -moz-opacity:1; filter:alpha(opacity=100)',
-      );
-    });
-  };
-
   /**
    * This method is executed after the render method is executed
    */
@@ -2692,8 +2673,8 @@ class MenuWidget extends React.Component {
     if (
       this.map.findLayerById(layer) &&
       this.map.findLayerById(layer) !== null &&
-      this.map.findLayerById(layer).opacity &&
-      this.map.findLayerById(layer).opacity !== null
+      (this.map.findLayerById(layer).opacity ||
+        this.map.findLayerById(layer).opacity === 0)
     ) {
       this.map.findLayerById(layer).opacity = value / 100;
     }
@@ -2701,7 +2682,6 @@ class MenuWidget extends React.Component {
       '.active-layer[layer-id="' + layer + '"] .active-layer-opacity',
     ).dataset.opacity = value;
     // }
-    this.setLegendOpacity();
   }
 
   /**
@@ -2816,16 +2796,15 @@ class MenuWidget extends React.Component {
       if (this.visibleLayers[elem.id][1] === 'eye-slash') {
         this.map.findLayerById(elem.id).visible = false;
       } else {
-        let layerOpacityes = JSON.parse(
+        let layerOpacities = JSON.parse(
           sessionStorage.getItem('layerOpacities'),
         );
         if (
-          layerOpacityes &&
-          layerOpacityes !== null &&
-          layerOpacityes[elem.id] &&
-          layerOpacityes[elem.id] !== null
+          layerOpacities &&
+          layerOpacities !== null &&
+          (layerOpacities[elem.id] || layerOpacities[elem.id] === 0)
         ) {
-          this.map.findLayerById(elem.id).opacity = layerOpacityes[elem.id];
+          this.map.findLayerById(elem.id).opacity = layerOpacities[elem.id];
         }
       }
     }