From 3542c38472ac27f6feb6fdbd4e34764e7d35fee3 Mon Sep 17 00:00:00 2001
From: diana-villalvazo-wgu <diana.villalvazo@wgu.edu>
Date: Mon, 12 May 2025 16:33:36 -0600
Subject: [PATCH] fix: incorrect usage of header elements

---
 src/containers/CoursesPanel/NoCoursesView/index.jsx       | 4 ++--
 src/containers/Dashboard/DashboardLayout.jsx              | 3 +--
 src/containers/Dashboard/index.scss                       | 8 ++++++++
 .../LearnerDashboardHeader/ConfirmEmailBanner/index.jsx   | 2 +-
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/src/containers/CoursesPanel/NoCoursesView/index.jsx b/src/containers/CoursesPanel/NoCoursesView/index.jsx
index 0810e04..f7a38c6 100644
--- a/src/containers/CoursesPanel/NoCoursesView/index.jsx
+++ b/src/containers/CoursesPanel/NoCoursesView/index.jsx
@@ -19,9 +19,9 @@ export const NoCoursesView = () => {
       className="d-flex align-items-center justify-content-center mb-4.5"
     >
       <Image src={emptyCourseSVG} alt={formatMessage(messages.bannerAlt)} />
-      <h1>
+      <h3 className="h1">
         {formatMessage(messages.lookingForChallengePrompt)}
-      </h1>
+      </h3>
       <p>
         {formatMessage(messages.exploreCoursesPrompt)}
       </p>
diff --git a/src/containers/Dashboard/DashboardLayout.jsx b/src/containers/Dashboard/DashboardLayout.jsx
index d970998..4c68236 100644
--- a/src/containers/Dashboard/DashboardLayout.jsx
+++ b/src/containers/Dashboard/DashboardLayout.jsx
@@ -40,8 +40,7 @@ export const DashboardLayout = ({ children }) => {
         <Col {...courseListColumnProps} className="course-list-column">
           {children}
         </Col>
-        <Col {...columnConfig.sidebar} className="sidebar-column">
-          {!isCollapsed && (<h2 className="course-list-title">&nbsp;</h2>)}
+        <Col {...columnConfig.sidebar} className={['sidebar-column', !isCollapsed && 'not-collapsed']}>
           <WidgetSidebarSlot />
         </Col>
       </Row>
diff --git a/src/containers/Dashboard/index.scss b/src/containers/Dashboard/index.scss
index 9b5089f..be09d46 100644
--- a/src/containers/Dashboard/index.scss
+++ b/src/containers/Dashboard/index.scss
@@ -6,6 +6,14 @@
 
 .sidebar-column {
   padding: 0 map-get($spacers, 3) 0 map-get($spacers, 1);
+
+  &.not-collapsed {
+    padding-top: map-get($spacers, 2);
+
+    & >:first-child {
+      margin-top: map-get($spacers, 5\.5);
+    }
+  }
 }
 
 @include media-breakpoint-down(lg) {
diff --git a/src/containers/LearnerDashboardHeader/ConfirmEmailBanner/index.jsx b/src/containers/LearnerDashboardHeader/ConfirmEmailBanner/index.jsx
index f5f9313..a5eb1fa 100644
--- a/src/containers/LearnerDashboardHeader/ConfirmEmailBanner/index.jsx
+++ b/src/containers/LearnerDashboardHeader/ConfirmEmailBanner/index.jsx
@@ -64,7 +64,7 @@ export const ConfirmEmailBanner = () => {
           </Button>
         )}
       >
-        <h1 className="text-center p-3">{formatMessage(messages.confirmEmailModalHeader)}</h1>
+        <h4 className="text-center p-3 h1">{formatMessage(messages.confirmEmailModalHeader)}</h4>
         <p className="text-center">{formatMessage(messages.confirmEmailModalBody)}</p>
       </MarketingModal>
     </>