From 7a37861b17ff9e8bf38d647e1ba83a66d5a7cee6 Mon Sep 17 00:00:00 2001
From: Stephanie Leary <stephanie.leary@equinoxoli.org>
Date: Fri, 14 Apr 2023 19:27:26 +0000
Subject: [PATCH] LP1994711 Semantic H1 headings for page titles

Patch alters the markup within <eg-staff-banner> to include H1 headings
for page titles.

The patch also includes a new CSS file for the course page component,
which appears to be the only component taking advantage of the custom
classes and icons for page titles. I've made archived course titles gray
and italicized.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
---
 .../app/staff/admin/local/course-reserves/course-list.component.ts | 3 ++-
 .../staff/admin/local/course-reserves/course-page.component.css    | 4 ++++
 .../staff/admin/local/course-reserves/course-page.component.html   | 2 +-
 Open-ILS/src/eg2/src/app/staff/share/staff-banner.component.ts     | 7 ++++---
 Open-ILS/src/eg2/src/styles.css                                    | 5 +++++
 5 files changed, 16 insertions(+), 5 deletions(-)
 create mode 100644 Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.css

diff --git a/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-list.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-list.component.ts
index cc1f75b51e..a5b4f0d124 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-list.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-list.component.ts
@@ -21,7 +21,8 @@ import {CourseAssociateUsersComponent
     } from './course-associate-users.component';
 
 @Component({
-    templateUrl: './course-list.component.html'
+    templateUrl: './course-list.component.html',
+    styleUrls: ['./course-page.component.css']
 })
 
 export class CourseListComponent implements OnInit, AfterViewInit {
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.css b/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.css
new file mode 100644
index 0000000000..0beaeb7616
--- /dev/null
+++ b/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.css
@@ -0,0 +1,4 @@
+.course-archived {
+  color: #666;
+  font-style: italic;
+}
\ No newline at end of file
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.html b/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.html
index d8bae1bc6e..fe001ef231 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-page.component.html
@@ -1,7 +1,7 @@
 <eg-staff-banner
   bannerText=" {{currentCourse.course_number()}}: {{currentCourse.name()}}"
   i18n-bannerText class="mb-3" *ngIf="currentCourse"
-  [bannerStyle]="(courseIsArchived || currentCourse.is_archived()) === 't' ? 'alert-secondary' : null"
+  [bannerStyle]="(courseIsArchived || currentCourse.is_archived()) === 't' ? 'course-archived' : null"
   [bannerIcon]="(courseIsArchived || currentCourse.is_archived()) === 't' ? 'lock' : null">
 </eg-staff-banner>
 
diff --git a/Open-ILS/src/eg2/src/app/staff/share/staff-banner.component.ts b/Open-ILS/src/eg2/src/app/staff/share/staff-banner.component.ts
index c49c495ea9..6055fbcaa3 100644
--- a/Open-ILS/src/eg2/src/app/staff/share/staff-banner.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/share/staff-banner.component.ts
@@ -5,9 +5,10 @@ import {Component, OnInit, Input} from '@angular/core';
   template: `
     <div class="lead alert alert-primary text-center pt-1 pb-1"
       [ngClass]="bannerStyle ? bannerStyle : 'alert-primary'">
-      <eg-title i18n-prefix [prefix]="bannerText"></eg-title>
-      <i class="material-icons align-middle text-left" *ngIf="bannerIcon">{{bannerIcon}}</i>
-       <span class="align-middle">{{bannerText}}</span>
+      <h1>
+        <i class="material-icons align-middle text-left" *ngIf="bannerIcon">{{bannerIcon}}</i>
+        <span i18n>{{bannerText}}</span>
+      </h1>
     </div>
     `
 })
diff --git a/Open-ILS/src/eg2/src/styles.css b/Open-ILS/src/eg2/src/styles.css
index 58c97bfe7a..98abe7779c 100644
--- a/Open-ILS/src/eg2/src/styles.css
+++ b/Open-ILS/src/eg2/src/styles.css
@@ -23,6 +23,11 @@ body, .form-control, .btn, .input-group-text {
    */
   font-size: .88rem;
 }
+h1 {
+  font-size: 1.25rem;
+  font-weight: normal;
+  margin: 0
+}
 h2 {
   font-size: 1.25rem;
   font-weight: 550;
-- 
2.11.0