From 9d56474197910ae874980d413d868ea89c4a5255 Mon Sep 17 00:00:00 2001
From: PhilippeL <PHILIPPE.LAFRENIERE@msp.gouv.qc.ca>
Date: Mon, 4 May 2020 15:57:10 -0400
Subject: [PATCH] fix(layer-list): fix baselayer with select all and opacity
 CSS

---
 .../layer/layer-item/layer-item.component.ts  |  8 ++----
 .../layer-list/layer-list.component.html      | 28 ++++++++++---------
 .../layer/layer-list/layer-list.component.ts  |  8 ++++++
 packages/geo/src/lib/map/shared/map.ts        |  8 ++++--
 4 files changed, 32 insertions(+), 20 deletions(-)

diff --git a/packages/geo/src/lib/layer/layer-item/layer-item.component.ts b/packages/geo/src/lib/layer/layer-item/layer-item.component.ts
index 29a28fac9d..eb28de6066 100644
--- a/packages/geo/src/lib/layer/layer-item/layer-item.component.ts
+++ b/packages/geo/src/lib/layer/layer-item/layer-item.component.ts
@@ -58,17 +58,15 @@ export class LayerItemComponent implements OnInit, OnDestroy {
   }
   set selectAll(value: boolean) {
     this._selectAll = value;
-    if (value) {
-      if (!this.layer.baseLayer) {
-        this.layerCheck = true;
-      }
+    if (value === true) {
+      this.layerCheck = true;
     } else {
       this.layerCheck = false;
     }
   }
   private _selectAll = false;
 
-  public layerCheck = false;
+  public layerCheck;
 
   private resolution$$: Subscription;
 
diff --git a/packages/geo/src/lib/layer/layer-list/layer-list.component.html b/packages/geo/src/lib/layer/layer-list/layer-list.component.html
index c8ca50b745..4c433eeebf 100644
--- a/packages/geo/src/lib/layer/layer-list/layer-list.component.html
+++ b/packages/geo/src/lib/layer/layer-list/layer-list.component.html
@@ -129,19 +129,21 @@
     </button>
 
     <mat-menu #opacityMenu="matMenu">
-      <mat-slider *ngIf="layersChecked.length"
-        id="opacity-slider"
-        color="primary"
-        thumbLabel
-        tickInterval="5"
-        step="5"
-        [min]="0"
-        [max]="100"
-        [(ngModel)]="checkOpacity"
-        [matTooltip]="'igo.geo.layer.opacity' | translate"
-        matTooltipShowDelay="500"
-        tooltip-position="below">
-      </mat-slider>
+      <div id="opacity-menu">
+        <mat-slider *ngIf="layersChecked.length"
+          id="opacity-slider"
+          color="primary"
+          thumbLabel
+          tickInterval="5"
+          step="5"
+          [min]="0"
+          [max]="100"
+          [(ngModel)]="checkOpacity"
+          [matTooltip]="'igo.geo.layer.opacity' | translate"
+          matTooltipShowDelay="500"
+          tooltip-position="below">
+        </mat-slider>
+      </div>
     </mat-menu>
 
     <button
diff --git a/packages/geo/src/lib/layer/layer-list/layer-list.component.ts b/packages/geo/src/lib/layer/layer-list/layer-list.component.ts
index 28118ba55d..bb4209f0f9 100644
--- a/packages/geo/src/lib/layer/layer-list/layer-list.component.ts
+++ b/packages/geo/src/lib/layer/layer-list/layer-list.component.ts
@@ -237,6 +237,10 @@ export class LayerListComponent implements OnInit, OnDestroy {
         response = true;
       }
     }
+
+    if (this.layersChecked.length === 1 && this.layersChecked[0].baseLayer) {
+      response = false;
+    }
     return response;
   }
 
@@ -249,6 +253,10 @@ export class LayerListComponent implements OnInit, OnDestroy {
         response = true;
       }
     }
+
+    if (this.layersChecked.length === 1 && this.layersChecked[0].baseLayer) {
+      response = false;
+    }
     return response;
   }
 
diff --git a/packages/geo/src/lib/map/shared/map.ts b/packages/geo/src/lib/map/shared/map.ts
index 675ae0f073..8d94119652 100644
--- a/packages/geo/src/lib/map/shared/map.ts
+++ b/packages/geo/src/lib/map/shared/map.ts
@@ -279,7 +279,9 @@ export class IgoMap {
 
   raiseLayers(layers: Layer[]) {
     for (const layer of layers) {
-      this.raiseLayer(layer);
+      if (layer.baseLayer !== true) {
+        this.raiseLayer(layer);
+      }
     }
   }
 
@@ -293,7 +295,9 @@ export class IgoMap {
   lowerLayers(layers: Layer[]) {
     const reverseLayers = layers.reverse();
     for (const layer of reverseLayers) {
-      this.lowerLayer(layer);
+      if (layer.baseLayer !== true) {
+        this.lowerLayer(layer);
+      }
     }
   }