From d56bb72819c5c3037ad1aa1cccecf91b1a26cf3c Mon Sep 17 00:00:00 2001
From: Martin Brennan <martin@discourse.org>
Date: Thu, 10 Oct 2024 17:18:26 +1000
Subject: [PATCH] UX: Change AdminPageSubheader to H2 (#29149)

Since the AdminPageHeader is H1, it is more semantically
correct to progress to H2 after it rather than skipping
a level to H3

Also amend style of H2 to make it the same size as H3
---
 .../admin/addon/components/admin-page-subheader.gjs         | 2 +-
 app/assets/stylesheets/common/admin/admin_page_header.scss  | 6 +++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/admin/addon/components/admin-page-subheader.gjs b/app/assets/javascripts/admin/addon/components/admin-page-subheader.gjs
index c2d10775b94..642bfb3e99d 100644
--- a/app/assets/javascripts/admin/addon/components/admin-page-subheader.gjs
+++ b/app/assets/javascripts/admin/addon/components/admin-page-subheader.gjs
@@ -38,7 +38,7 @@ export default class AdminPageSubheader extends Component {
   <template>
     <div class="admin-page-subheader">
       <div class="admin-page-subheader__title-row">
-        <h3 class="admin-page-subheader__title">{{this.title}}</h3>
+        <h2 class="admin-page-subheader__title">{{this.title}}</h2>
         {{#if (has-block "actions")}}
           <div class="admin-page-subheader__actions">
             {{#if this.site.mobileView}}
diff --git a/app/assets/stylesheets/common/admin/admin_page_header.scss b/app/assets/stylesheets/common/admin/admin_page_header.scss
index c29fdb31e1b..fa8be9cfab1 100644
--- a/app/assets/stylesheets/common/admin/admin_page_header.scss
+++ b/app/assets/stylesheets/common/admin/admin_page_header.scss
@@ -7,10 +7,14 @@
     margin-bottom: var(--space-2);
 
     h1,
-    h3 {
+    h2 {
       margin: 0;
     }
 
+    h2 {
+      font-size: var(--font-up-2);
+    }
+
     .admin-page-header__actions {
       display: flex;
       align-items: center;