From 6c663d72ae1800c9c9a09276c2925ab111e32089 Mon Sep 17 00:00:00 2001
From: Alec Li <smartspot2@gmail.com>
Date: Mon, 30 Oct 2023 00:53:32 -0700
Subject: [PATCH] Fix outline icons

---
 csm_web/frontend/src/components/App.tsx                      | 2 +-
 .../enrollment_automation/coordinator/ReleaseStage.tsx       | 2 +-
 csm_web/frontend/src/css/base/base.scss                      | 5 +++++
 csm_web/frontend/src/css/header.scss                         | 4 ++--
 csm_web/frontend/src/css/section.scss                        | 4 ++--
 5 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/csm_web/frontend/src/components/App.tsx b/csm_web/frontend/src/components/App.tsx
index 1c321f7a..841364f2 100644
--- a/csm_web/frontend/src/components/App.tsx
+++ b/csm_web/frontend/src/components/App.tsx
@@ -141,7 +141,7 @@ function Header(): React.ReactElement {
           <h3 className="site-subtitle">Policies</h3>
         </NavLink>
         <a id="logout-btn" href="/logout" title="Log out">
-          <LogOutIcon width="1.25em" height="1.25em" />
+          <LogOutIcon className="icon" />
         </a>
       </div>
     </header>
diff --git a/csm_web/frontend/src/components/enrollment_automation/coordinator/ReleaseStage.tsx b/csm_web/frontend/src/components/enrollment_automation/coordinator/ReleaseStage.tsx
index 244bc58a..79842559 100644
--- a/csm_web/frontend/src/components/enrollment_automation/coordinator/ReleaseStage.tsx
+++ b/csm_web/frontend/src/components/enrollment_automation/coordinator/ReleaseStage.tsx
@@ -548,7 +548,7 @@ function MentorListItem({
     <div className={classString}>
       {removeIcon}
       <EyeIcon
-        className="icon mentor-list-icon mentor-list-item-view"
+        className="icon outline mentor-list-icon mentor-list-item-view"
         onClick={() => hasPreferences && setSelectedMentor(mentor)}
       />
       <div className="mentor-list-item-name">{mentor.name}</div>
diff --git a/csm_web/frontend/src/css/base/base.scss b/csm_web/frontend/src/css/base/base.scss
index b2532d93..c156bed2 100644
--- a/csm_web/frontend/src/css/base/base.scss
+++ b/csm_web/frontend/src/css/base/base.scss
@@ -66,6 +66,11 @@ h6 {
   stroke: currentcolor;
 }
 
+// some icons are outline icons that require no fill
+.icon.outline {
+  fill: none;
+}
+
 /// Styling for relation label; used across multiple pages
 .relation-label {
   min-width: 60px;
diff --git a/csm_web/frontend/src/css/header.scss b/csm_web/frontend/src/css/header.scss
index 7b45b3b4..c3674469 100644
--- a/csm_web/frontend/src/css/header.scss
+++ b/csm_web/frontend/src/css/header.scss
@@ -60,12 +60,12 @@ header {
   margin: auto 5px auto 30px;
 }
 
-#logout-btn * {
+#logout-btn .icon {
   vertical-align: middle;
   fill: #808080;
 }
 
-#logout-btn:hover * {
+#logout-btn:hover .icon {
   fill: #5e5e5e;
 }
 
diff --git a/csm_web/frontend/src/css/section.scss b/csm_web/frontend/src/css/section.scss
index 6afc36eb..97b4e801 100644
--- a/csm_web/frontend/src/css/section.scss
+++ b/csm_web/frontend/src/css/section.scss
@@ -213,6 +213,7 @@
 }
 
 #copy-student-emails {
+  margin-right: 8px;
   cursor: pointer;
 }
 
@@ -382,8 +383,7 @@
 #attendance-date-tabs-container {
   display: flex;
   margin: 0 -40px;
-  overflow-x: scroll;
-  overflow-y: hidden;
+  overflow: scroll hidden;
 }
 
 #attendance-date-tabs-container > div {