From 919e5fb44bb55e33aba06d438ad04c405e4abecf Mon Sep 17 00:00:00 2001
From: Piotr Pekala <piotrpawelpekala@gmail.com>
Date: Tue, 26 Nov 2019 08:06:07 -0800
Subject: [PATCH] Theme for mat icons updated

---
 .../nodes-menu/nodes-menu.component.html         | 10 +++++-----
 .../nodes-menu/nodes-menu.component.scss         |  8 ++++++++
 .../project-map-menu.component.html              | 16 ++++++----------
 .../project-map-menu.component.scss              |  8 ++++++++
 .../project-map/project-map.component.html       |  4 ++--
 .../project-map/project-map.component.scss       |  8 ++++++++
 src/theme-light.scss                             |  7 -------
 7 files changed, 37 insertions(+), 24 deletions(-)

diff --git a/src/app/components/project-map/nodes-menu/nodes-menu.component.html b/src/app/components/project-map/nodes-menu/nodes-menu.component.html
index aab5972f..9f3b04ef 100644
--- a/src/app/components/project-map/nodes-menu/nodes-menu.component.html
+++ b/src/app/components/project-map/nodes-menu/nodes-menu.component.html
@@ -4,7 +4,7 @@
     (click)="startConsoleForAllNodes()"
     class="menu-button"
     >
-    <mat-icon>web_asset</mat-icon>
+    <mat-icon class="unmarked">web_asset</mat-icon>
 </button>
 <button
     matTooltip="Start/Resume all nodes"
@@ -12,7 +12,7 @@
     (click)="startNodes()"
     class="menu-button"
     >
-    <mat-icon>play_arrow</mat-icon>
+    <mat-icon class="unmarked">play_arrow</mat-icon>
 </button>
 <button
     matTooltip="Suspend all nodes"
@@ -20,7 +20,7 @@
     (click)="suspendNodes()"
     class="menu-button"
     >
-    <mat-icon>pause</mat-icon>
+    <mat-icon class="unmarked">pause</mat-icon>
 </button>
 <button
     matTooltip="Stop all nodes"
@@ -28,7 +28,7 @@
     (click)="stopNodes()"
     class="menu-button"
     >
-    <mat-icon>stop</mat-icon>
+    <mat-icon class="unmarked">stop</mat-icon>
 </button>
 <button
     matTooltip="Reload all nodes"
@@ -36,5 +36,5 @@
     (click)="reloadNodes()"
     class="menu-button"
     >
-    <mat-icon>replay</mat-icon>
+    <mat-icon class="unmarked">replay</mat-icon>
 </button>
diff --git a/src/app/components/project-map/nodes-menu/nodes-menu.component.scss b/src/app/components/project-map/nodes-menu/nodes-menu.component.scss
index cfac8782..79ef3a63 100644
--- a/src/app/components/project-map/nodes-menu/nodes-menu.component.scss
+++ b/src/app/components/project-map/nodes-menu/nodes-menu.component.scss
@@ -10,3 +10,11 @@
     border: none;
     background-color: transparent;
 }
+
+.unmarked {
+    color: white!important;
+}
+
+.marked {
+    color: #0097a7!important;
+}
diff --git a/src/app/components/project-map/project-map-menu/project-map-menu.component.html b/src/app/components/project-map/project-map-menu/project-map-menu.component.html
index 4d099f56..985ba085 100644
--- a/src/app/components/project-map/project-map-menu/project-map-menu.component.html
+++ b/src/app/components/project-map/project-map-menu/project-map-menu.component.html
@@ -2,9 +2,8 @@
     matTooltip="Add a note"
     mat-icon-button
     class="menu-button"
-    [color]="drawTools.isTextChosen ? 'primary' : 'basic'"
     (click)="addDrawing('text')">
-    <mat-icon>create</mat-icon>
+    <mat-icon [ngClass]="{unmarked: !drawTools.isTextChosen, marked: drawTools.isTextChosen}">create</mat-icon>
 </button>
 <input
     type="file"
@@ -17,23 +16,21 @@
     mat-icon-button
     class="menu-button"
     (click)="file.click()">
-    <mat-icon>image</mat-icon>
+    <mat-icon class="unmarked">image</mat-icon>
 </button>
 <button
     matTooltip="Draw a rectangle"
     mat-icon-button
     class="menu-button"
-    [color]="drawTools.isRectangleChosen ? 'primary' : 'basic'"
     (click)="addDrawing('rectangle')">
-    <mat-icon>crop_3_2</mat-icon>
+    <mat-icon [ngClass]="{unmarked: !drawTools.isRectangleChosen, marked: drawTools.isRectangleChosen}">crop_3_2</mat-icon>
 </button>
 <button
     matTooltip="Draw an ellipse"
     mat-icon-button
     class="menu-button"
-    [color]="drawTools.isEllipseChosen ? 'primary' : 'basic'"
     (click)="addDrawing('ellipse')">
-    <mat-icon>panorama_fish_eye</mat-icon>
+    <mat-icon [ngClass]="{unmarked: !drawTools.isEllipseChosen, marked: drawTools.isEllipseChosen}">panorama_fish_eye</mat-icon>
 </button>
 <button 
     matTooltip="Draw a line" 
@@ -53,9 +50,8 @@
     matTooltip="Lock or unlock all items"
     mat-icon-button
     class="menu-button"
-    [color]="isLocked ? 'primary' : 'basic'"
     (click)="changeLockValue()">
-    <mat-icon>lock</mat-icon>
+    <mat-icon [ngClass]="{unmarked: !isLocked, marked: isLocked}">lock</mat-icon>
 </button>
 <button
     matTooltip="Take a screenshot"
@@ -63,7 +59,7 @@
     class="menu-button"
     (click)="takeScreenshot()"
     >
-    <mat-icon>photo_camera</mat-icon>
+    <mat-icon class="unmarked">photo_camera</mat-icon>
 </button>
 <app-drawing-added
   [server]="server"
diff --git a/src/app/components/project-map/project-map-menu/project-map-menu.component.scss b/src/app/components/project-map/project-map-menu/project-map-menu.component.scss
index f7626cab..133cf06e 100644
--- a/src/app/components/project-map/project-map-menu/project-map-menu.component.scss
+++ b/src/app/components/project-map/project-map-menu/project-map-menu.component.scss
@@ -22,3 +22,11 @@ mat-divider.divider {
 .non-visible {
     display: none;
 }
+
+.unmarked {
+    color: white!important;
+}
+
+.marked {
+    color: #0097a7!important;
+}
diff --git a/src/app/components/project-map/project-map.component.html b/src/app/components/project-map/project-map.component.html
index 28d3cee4..4b8b52a3 100644
--- a/src/app/components/project-map/project-map.component.html
+++ b/src/app/components/project-map/project-map.component.html
@@ -146,14 +146,14 @@
   </div>
 
   <div id="show-menu-wrapper" [ngClass]="{ shadowed: !isProjectMapMenuVisible }" *ngIf="!readonly">
-    <button class="arrow-button" mat-icon-button (click)="showMenu()"><mat-icon>keyboard_arrow_right</mat-icon></button>
+    <button class="arrow-button" mat-icon-button (click)="showMenu()"><mat-icon class="unmarked">keyboard_arrow_right</mat-icon></button>
   </div>
 
   <div id="menu-wrapper" [ngClass]="{ extended: isProjectMapMenuVisible }">
     <app-nodes-menu [server]="server" [project]="project"></app-nodes-menu>
     <mat-divider class="divider" [vertical]="true"></mat-divider>
     <app-project-map-menu [server]="server" [project]="project"></app-project-map-menu>
-    <button class="arrow-button" mat-icon-button (click)="hideMenu()"><mat-icon>keyboard_arrow_left</mat-icon></button>
+    <button class="arrow-button" mat-icon-button (click)="hideMenu()"><mat-icon class="unmarked">keyboard_arrow_left</mat-icon></button>
   </div>
 
   <app-context-menu [project]="project" [server]="server"></app-context-menu>
diff --git a/src/app/components/project-map/project-map.component.scss b/src/app/components/project-map/project-map.component.scss
index e7458af0..20465534 100644
--- a/src/app/components/project-map/project-map.component.scss
+++ b/src/app/components/project-map/project-map.component.scss
@@ -93,6 +93,14 @@ mat-divider.divider {
   color: gray;
 }
 
+.unmarked {
+  color: white!important;
+}
+
+.marked {
+  color: #0097a7!important;
+}
+
 #zoom-buttons {
   position: fixed;
   background: #263238;
diff --git a/src/theme-light.scss b/src/theme-light.scss
index 65e23d82..29585d89 100644
--- a/src/theme-light.scss
+++ b/src/theme-light.scss
@@ -38,13 +38,6 @@ $theme: map-merge(
   )
 );
 
-@mixin mat-button-theme($theme) {
-  .mat-button, .mat-icon-button, .mat-stroked-button {
-    color: white;
-    background: transparent;
-  }
-}
-
 @include mat-core-theme($theme);
 @include mat-autocomplete-theme($theme);
 @include mat-button-theme($theme);