From d8725d7c18a1a6fc1d5e9f2f3afa6b2ed5ece167 Mon Sep 17 00:00:00 2001
From: Arunmozhi <arunmozhi@opencraft.com>
Date: Mon, 6 May 2024 21:01:12 +1000
Subject: [PATCH] fix: style buttons consistently; adds dashboard in navbar

---
 src/components/course/SubsidyRequestButton.jsx        |  6 ++++--
 .../dashboard/main-content/DashboardMainContent.jsx   |  2 +-
 src/components/site-header/SiteHeaderNavMenu.jsx      | 11 ++++++++---
 src/components/site-header/styles/_Header.scss        |  5 +++++
 4 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/src/components/course/SubsidyRequestButton.jsx b/src/components/course/SubsidyRequestButton.jsx
index 830c35fac4..4edb605e6d 100644
--- a/src/components/course/SubsidyRequestButton.jsx
+++ b/src/components/course/SubsidyRequestButton.jsx
@@ -18,7 +18,7 @@ const props = {
     requested: 'Awaiting approval',
   },
   disabledStates: ['requested'],
-  variant: 'primary',
+  variant: 'outline-primary',
   className: 'mb-4 mt-1',
 };
 
@@ -114,7 +114,9 @@ const SubsidyRequestButton = () => {
   };
 
   return (
-    <StatefulButton {...props} state={getButtonState()} onClick={handleRequestButtonClick} />
+    <div>
+      <StatefulButton {...props} state={getButtonState()} onClick={handleRequestButtonClick} />
+    </div>
   );
 };
 
diff --git a/src/components/dashboard/main-content/DashboardMainContent.jsx b/src/components/dashboard/main-content/DashboardMainContent.jsx
index 057b8db140..fa7bfb7908 100644
--- a/src/components/dashboard/main-content/DashboardMainContent.jsx
+++ b/src/components/dashboard/main-content/DashboardMainContent.jsx
@@ -37,7 +37,7 @@ const DashboardMainContent = ({ canOnlyViewHighlightSets }) => {
           <img src={CourseSearchIcon} alt="Search Courses Icon" />
           <h4 className="h3 my-5">Enroll in Learning Paths or Courses</h4>
           <p>
-            <Button as={Link} to={`/${slug}/search`} variant="primary">Explore Now</Button>
+            <Button as={Link} to={`/${slug}/search`} variant="outline-primary">Explore Now</Button>
           </p>
         </div>
       </CourseEnrollments>
diff --git a/src/components/site-header/SiteHeaderNavMenu.jsx b/src/components/site-header/SiteHeaderNavMenu.jsx
index 2d071370c4..b1f1e90dd8 100644
--- a/src/components/site-header/SiteHeaderNavMenu.jsx
+++ b/src/components/site-header/SiteHeaderNavMenu.jsx
@@ -11,9 +11,14 @@ const SiteHeaderNavMenu = () => {
   }
 
   return (
-    <NavLink to={`/${enterpriseConfig.slug}/search`} className={mainMenuLinkClassName} exact>
-      Course Finder
-    </NavLink>
+    <>
+      <NavLink to={`/${enterpriseConfig.slug}`} className={mainMenuLinkClassName} exact>
+        Dashboard
+      </NavLink>
+      <NavLink to={`/${enterpriseConfig.slug}/search`} className={mainMenuLinkClassName} exact>
+        Course Finder
+      </NavLink>
+    </>
   );
 };
 
diff --git a/src/components/site-header/styles/_Header.scss b/src/components/site-header/styles/_Header.scss
index d8327c9dcd..d571a26061 100644
--- a/src/components/site-header/styles/_Header.scss
+++ b/src/components/site-header/styles/_Header.scss
@@ -53,6 +53,11 @@ $header-logo-height-mobile: 1.5rem;
   background: $white;
   .nav-link {
     text-decoration: none;
+    border-radius: $border-radius;
+    &:hover {
+      background-color: $primary;
+      color: $white;
+    }
   }
   .logo,
   .vertical-divider {