(trafficcontrol) branch master updated: TPv2 Add page title setting when header title is updated (#7915)

[zrhoffman]

This is an automated email from the ASF dual-hosted git repository.

zrhoffman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new cb47ce8870 TPv2 Add page title setting when header title is updated 
(#7915)
cb47ce8870 is described below

commit cb47ce8870a76f4e5a27eae9ccd1b39fa1e9001a
Author: ocket8888 <ocket8...@apache.org>
AuthorDate: Fri Jan 19 10:21:47 2024 -0700

    TPv2 Add page title setting when header title is updated (#7915)
    
    Add page title setting when header title is updated
---
 .../app/shared/navigation/navigation.service.spec.ts  | 18 ++++++++++--------
 .../src/app/shared/navigation/navigation.service.ts   | 19 +++++++++++++------
 2 files changed, 23 insertions(+), 14 deletions(-)

diff --git 
a/experimental/traffic-portal/src/app/shared/navigation/navigation.service.spec.ts
 
b/experimental/traffic-portal/src/app/shared/navigation/navigation.service.spec.ts
index 7ba4122312..9b5d52afd5 100644
--- 
a/experimental/traffic-portal/src/app/shared/navigation/navigation.service.spec.ts
+++ 
b/experimental/traffic-portal/src/app/shared/navigation/navigation.service.spec.ts
@@ -15,13 +15,14 @@ import { HttpClientModule } from "@angular/common/http";
 import { TestBed } from "@angular/core/testing";
 import { MatButtonModule } from "@angular/material/button";
 import { MatMenuModule } from "@angular/material/menu";
+import { Title } from "@angular/platform-browser";
 import { RouterTestingModule } from "@angular/router/testing";
 import { of } from "rxjs";
 
 import { UserService } from "src/app/api";
 import { APITestingModule } from "src/app/api/testing";
 import { CurrentUserService } from 
"src/app/shared/current-user/current-user.service";
-import { HeaderNavigation, NavigationService } from 
"src/app/shared/navigation/navigation.service";
+import { NavigationService } from 
"src/app/shared/navigation/navigation.service";
 import { type TpHeaderComponent } from 
"src/app/shared/navigation/tp-header/tp-header.component";
 
 describe("NavigationService", () => {
@@ -64,14 +65,15 @@ describe("NavigationService", () => {
                expect(logOutSpy).toHaveBeenCalled();
        });
 
-       it("set header component", () => {
+       it("sets the page title and header", () => {
                expect(mockHeaderComp).toBeTruthy();
-               expect(mockHeaderComp?.hidden).toBeFalse();
-               expect(mockHeaderComp?.title).toBe("");
+               expect(mockHeaderComp.hidden).toBeFalse();
+               expect(mockHeaderComp.title).toBe("");
 
-               service.headerHidden.next(true);
-               service.headerTitle.next("something else");
-               service.horizontalNavsUpdated.next(new 
Array<HeaderNavigation>());
-               service.verticalNavsUpdated.next(new Array<HeaderNavigation>());
+               const title = "something else";
+               const titleService = TestBed.inject(Title);
+
+               service.headerTitle.next(title);
+               expect(titleService.getTitle()).toBe(title);
        });
 });
diff --git 
a/experimental/traffic-portal/src/app/shared/navigation/navigation.service.ts 
b/experimental/traffic-portal/src/app/shared/navigation/navigation.service.ts
index faa1d87a92..aa69fad15b 100644
--- 
a/experimental/traffic-portal/src/app/shared/navigation/navigation.service.ts
+++ 
b/experimental/traffic-portal/src/app/shared/navigation/navigation.service.ts
@@ -13,6 +13,7 @@
 */
 import { isPlatformBrowser } from "@angular/common";
 import { Inject, Injectable, PLATFORM_ID } from "@angular/core";
+import { Title } from "@angular/platform-browser";
 import { ReplaySubject } from "rxjs";
 
 import { UserService } from "src/app/api";
@@ -71,9 +72,21 @@ export class NavigationService {
                private readonly api: UserService,
                @Inject(PLATFORM_ID) private readonly platformId: object,
                private readonly log: LoggingService,
+               private readonly pageTitle: Title,
        ) {
+               this.horizontalNavsUpdated = new ReplaySubject(1);
+               this.verticalNavsUpdated = new ReplaySubject(1);
+               this.headerTitle = new ReplaySubject(1);
+               this.headerTitle.next("Welcome to Traffic Portal!");
+               this.headerHidden = new ReplaySubject(1);
+               this.headerHidden.next(false);
                if (isPlatformBrowser(this.platformId)) {
                        this.tpv1Url = 
window.localStorage.getItem(LOCAL_TPV1_URL) ?? this.tpv1Url;
+                       this.headerTitle.subscribe(
+                               title => {
+                                       this.pageTitle.setTitle(title);
+                               }
+                       );
                }
                this.horizontalNavs = new Map<string, HeaderNavigation>([
                        ["Home", {
@@ -108,12 +121,6 @@ export class NavigationService {
                                        type: "button"
                                }],
                ]);
-               this.horizontalNavsUpdated = new ReplaySubject(1);
-               this.verticalNavsUpdated = new ReplaySubject(1);
-               this.headerTitle = new ReplaySubject(1);
-               this.headerTitle.next("Welcome to Traffic Portal!");
-               this.headerHidden = new ReplaySubject(1);
-               this.headerHidden.next(false);
                this.horizontalNavsUpdated.next(this.buildHorizontalNavs());
                this.verticalNavsUpdated.next(this.buildVerticalNavs());