From 8cd0fb67381aa3e236016547ae7e35e283b26c96 Mon Sep 17 00:00:00 2001
From: Frederic Junod <frederic.junod@camptocamp.com>
Date: Mon, 6 May 2019 15:03:25 +0200
Subject: [PATCH 1/4] Don't show the layer config button if the layer is not
 activated

---
 contribs/gmf/src/layertree/desktop.scss | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/contribs/gmf/src/layertree/desktop.scss b/contribs/gmf/src/layertree/desktop.scss
index b14221172f20..46064557de06 100644
--- a/contribs/gmf/src/layertree/desktop.scss
+++ b/contribs/gmf/src/layertree/desktop.scss
@@ -62,10 +62,16 @@ gmf-layertree {
     padding: $half-app-margin;
   }
 
-  .gmf-layertree-group:hover,
-  .gmf-layertree-leaf:hover {
-    .gmf-layertree-right-buttons {
-      display: block;
+  .gmf-layertree-group, .gmf-layertree-leaf {
+    &:hover {
+      .gmf-layertree-right-buttons {
+        display: block;
+      }
+    }
+    &.off {
+      .extra-actions {
+        display: none;
+      }
     }
   }
 

From 6b234d2d5865cd92c9abd3b115b7cf8f0a2504ba Mon Sep 17 00:00:00 2001
From: Frederic Junod <frederic.junod@camptocamp.com>
Date: Tue, 7 May 2019 11:09:33 +0200
Subject: [PATCH 2/4] Remove unnecessary data-toggle for legends

---
 contribs/gmf/src/layertree/component.html | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/contribs/gmf/src/layertree/component.html b/contribs/gmf/src/layertree/component.html
index 4cbf0a457178..abc0bc94881a 100644
--- a/contribs/gmf/src/layertree/component.html
+++ b/contribs/gmf/src/layertree/component.html
@@ -162,7 +162,6 @@
             <i class="fa fa-th-list fa-fw"></i>
             <a
               ng-click="::gmfLayertreeCtrl.toggleNodeLegend('#gmf-layertree-node-' + layertreeCtrl.uid + '-legend'); popoverCtrl.dismissPopover()"
-              data-toggle="collapse"
               href="">
               {{'Show/hide legend'|translate}}
             </a>
@@ -222,7 +221,6 @@
   <a
     class="gmf-layertree-node-menu-togglelegend"
     ng-if="::gmfLayertreeCtrl.supportsLegend(layertreeCtrl)"
-    data-toggle="collapse"
     ng-click="::gmfLayertreeCtrl.toggleNodeLegend('#gmf-layertree-node-' + layertreeCtrl.uid + '-legend')"
     href="">
     <span
@@ -240,7 +238,6 @@
 
   <a
     title="{{'Hide legend'|translate}}"
-    data-toggle="collapse"
     ng-click="::gmfLayertreeCtrl.toggleNodeLegend('#gmf-layertree-node-' + layertreeCtrl.uid + '-legend')"
     href="">
     {{'Hide legend'|translate}}

From 7e29776d493b5b616f8a8892bea7588e62a96acb Mon Sep 17 00:00:00 2001
From: Frederic Junod <frederic.junod@camptocamp.com>
Date: Wed, 8 May 2019 13:38:04 +0200
Subject: [PATCH 3/4] Fix indentation

---
 contribs/gmf/src/search/component.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/contribs/gmf/src/search/component.js b/contribs/gmf/src/search/component.js
index 270ebb5552e5..21fa96a8904e 100644
--- a/contribs/gmf/src/search/component.js
+++ b/contribs/gmf/src/search/component.js
@@ -696,10 +696,10 @@ class SearchController {
    */
   filterAction_(action) {
     return (
-    /**
-         * @param {import("geojson").Feature} feature
-         * @return {boolean}
-         */
+      /**
+       * @param {import("geojson").Feature} feature
+       * @return {boolean}
+       */
       function(feature) {
         const properties = feature.properties || {};
         if (properties.actions) {

From 2dfd4945839279cfe38113b3ccc41afa2142b596 Mon Sep 17 00:00:00 2001
From: Frederic Junod <frederic.junod@camptocamp.com>
Date: Wed, 8 May 2019 14:58:48 +0200
Subject: [PATCH 4/4] Hide the legend if the layer is not visible

---
 contribs/gmf/src/layertree/common.scss    | 10 ++++++++--
 contribs/gmf/src/layertree/component.html |  2 +-
 contribs/gmf/src/layertree/component.js   |  7 ++++---
 3 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/contribs/gmf/src/layertree/common.scss b/contribs/gmf/src/layertree/common.scss
index 6676084c7388..f94c418baf32 100644
--- a/contribs/gmf/src/layertree/common.scss
+++ b/contribs/gmf/src/layertree/common.scss
@@ -162,8 +162,14 @@
     border: 0.06rem solid $main-bg-color;
     background-color: lighten($main-bg-color, 8%);
 
-    .off {
-      opacity: 0.5;
+    display: none;
+    &.show {
+      // legend is displayed
+      display: block;
+      &.off {
+        // legend is displayed but the layer is not visible
+        display: none;
+      }
     }
 
     a {
diff --git a/contribs/gmf/src/layertree/component.html b/contribs/gmf/src/layertree/component.html
index abc0bc94881a..d41d85459191 100644
--- a/contribs/gmf/src/layertree/component.html
+++ b/contribs/gmf/src/layertree/component.html
@@ -233,7 +233,7 @@
 
 <div
   ng-if="::!layertreeCtrl.isRoot && gmfLayertreeCtrl.getLegendsObject(layertreeCtrl) && layertreeCtrl.node.metadata.legend" id="gmf-layertree-node-{{::layertreeCtrl.uid}}-legend"
-  class="collapse gmf-layertree-legend"
+  class="gmf-layertree-legend"
   ng-class="[gmfLayertreeCtrl.getNodeState(layertreeCtrl), layertreeCtrl.node.metadata.isLegendExpanded ? 'show' : '']">
 
   <a
diff --git a/contribs/gmf/src/layertree/component.js b/contribs/gmf/src/layertree/component.js
index e93c527bf74a..6a66ff80f401 100644
--- a/contribs/gmf/src/layertree/component.js
+++ b/contribs/gmf/src/layertree/component.js
@@ -807,9 +807,10 @@ Controller.prototype.zoomToResolution = function(treeCtrl) {
  * @param {string} legendNodeId The DOM node legend id to toggle
  */
 Controller.prototype.toggleNodeLegend = function(legendNodeId) {
-  $(legendNodeId).toggle({
-    toggle: true
-  });
+  const div = document.querySelector(legendNodeId);
+  if (div) {
+    div.classList.toggle('show');
+  }
 };