Page Titles (#475)

* titles * roadmap names * Update site/src/pages/CoursePage/index.tsx Co-authored-by: Jacob Sommer <js0mmer@users.noreply.github.com> * Update site/src/component/Report/Reports.tsx Co-authored-by: Jacob Sommer <js0mmer@users.noreply.github.com> * Update site/src/component/Verify/Verify.tsx Co-authored-by: Jacob Sommer <js0mmer@users.noreply.github.com> --------- Co-authored-by: Jacob Sommer <js0mmer@users.noreply.github.com>
icssc · May 20, 2024 · 167080a · 167080a
1 parent 6234693
commit 167080a
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 2 deletions.
diff --git a/site/src/component/Report/Reports.tsx b/site/src/component/Report/Reports.tsx
@@ -43,6 +43,7 @@ const Reports: FC = () => {
 
   useEffect(() => {
     getData();
+    document.title = 'View Reports | PeterPortal';
   }, []);
 
   const acceptReports = async (reviewID: string) => {

diff --git a/site/src/component/Verify/Verify.tsx b/site/src/component/Verify/Verify.tsx
@@ -18,6 +18,7 @@ const Verify: FC = () => {
 
   useEffect(() => {
     getUnverifiedReviews();
+    document.title = 'Verify Reviews | PeterPortal';
   }, []);
 
   const verifyReview = async (reviewID: string) => {

diff --git a/site/src/pages/CoursePage/index.tsx b/site/src/pages/CoursePage/index.tsx
@@ -29,6 +29,7 @@ const CoursePage: FC = () => {
         if (course) {
           dispatch(setCourse(course as CourseGQLData));
           setError('');
+          document.title = `${courseGQLData.department + ' ' + courseGQLData.courseNumber} | PeterPortal`;
         } else {
           setError(`Course ${id} does not exist!`);
         }

diff --git a/site/src/pages/ProfessorPage/index.tsx b/site/src/pages/ProfessorPage/index.tsx
@@ -27,6 +27,7 @@ const ProfessorPage: FC = () => {
         if (professor) {
           dispatch(setProfessor(professor as ProfessorGQLData));
           setError('');
+          document.title = `${(professor as ProfessorGQLData).name} | PeterPortal`;
         } else {
           setError(`Professor ${id} does not exist!`);
         }

diff --git a/site/src/pages/ReviewsPage/index.tsx b/site/src/pages/ReviewsPage/index.tsx
@@ -20,6 +20,7 @@ const ReviewsPage: FC = () => {
 
   useEffect(() => {
     checkLoggedIn();
+    document.title = 'Your Reviews | PeterPortal';
   }, []);
 
   if (!loaded) {

diff --git a/site/src/pages/RoadmapPage/RoadmapMultiplan.tsx b/site/src/pages/RoadmapPage/RoadmapMultiplan.tsx
@@ -1,4 +1,4 @@
-import { FC, useState } from 'react';
+import { FC, useEffect, useState } from 'react';
 import { useAppDispatch, useAppSelector } from '../../store/hooks';
 import {
   addRoadmapPlan,
@@ -90,6 +90,10 @@ const RoadmapMultiplan: FC = () => {
     setEditIdx(-1);
   };
 
+  useEffect(() => {
+    document.title = `${allPlans.plans[currentPlanIndex].name} | PeterPortal`;
+  }, [currentPlanIndex]);
+
   return (
     <div className="multi-plan-selector">
       <Dropdown

diff --git a/site/src/pages/SearchPage/index.tsx b/site/src/pages/SearchPage/index.tsx
@@ -1,4 +1,4 @@
-import { FC } from 'react';
+import { FC, useEffect } from 'react';
 import { useParams } from 'react-router-dom';
 import './SearchPage.scss';
 import 'react-multi-carousel/lib/styles.css';
@@ -13,6 +13,11 @@ import { SearchIndex } from '../../types/types';
 
 const SearchPage: FC = () => {
   const { index = 'courses' } = useParams<{ index: SearchIndex }>();
+
+  useEffect(() => {
+    document.title = `${index === 'courses' ? 'Courses' : 'Professors'} | PeterPortal`;
+  }, [index]);
+
   return (
     <>
       <div id="content-container">