Skip to content

Add Footer #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
norioriroiron wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add Footer #9

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fd67f84
add footer
norioriroiron Jun 4, 2026
8636230
implement requested changes
norioriroiron Jun 6, 2026
67e8452
cleanup and format
norioriroiron Jun 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/App.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import '@xyflow/react/dist/style.css';
import Header from '@/components/Header';
import Explorer from '@/pages/Explorer';
import Planner from '@/pages/Planner';
import Footer from '@/components/Footer';

export default function App() {
return (
Expand All @@ -16,6 +17,7 @@ export default function App() {
<Route path="/planner" element={<Planner />} />
</Routes>
</main>
<Footer />
</div>
</BrowserRouter>
);
Expand Down
33 changes: 33 additions & 0 deletions src/components/Footer.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
export default function Footer() {
Copy link
Copy Markdown
Contributor

@AJaccP AJaccP Jun 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Content's all here which is great! A couple notes, mostly visual and about text hierarchy:

  • "Carleton Computer Science Society" at the start should be the most prominent part, so you can make the text bigger / bolder. Try using something like Tailwinds' text-lg font-semibold while the rest stays text-xs.
  • The copyright line at the end can also be slightly bigger than the disclaimer lines. Try using text-sm for that instead of text-xs)
  • Looking at the single column, it looks a bit tall. It might be worth making the GitHub repo link inline with one of the other lines, maybe the copyright one? So it looks like (© 2026 Carleton Computer Science Society · GitHub).
  • For consistency, use Tailwind + block elements instead of raw <b>, <u> or <br/> tags.
    • <b> → font-bold (or font-semibold)
    • <u> → underline
    • the <br/>s → wrap each line in its own <p> or <div> and let spacing handle the breaks, rather than manual line breaks

return (
<footer className="bg-black border-t-red-600 border-t-2 p-1">
<div className="text-center text-white text-xs">
<p className="text-lg font-semibold">
<a href="https://ccss.carleton.ca/">
Carleton Computer Science Society
</a>
</p>
<p>
Not affiliated with or endorsed by Carleton University. Always verify
course requirements in the{' '}
<a
className="underline"
href="https://calendar.carleton.ca/undergrad/undergradprograms/computerscience/"
>
official undergraduate calendar
</a>
.
</p>
<p className="text-sm">
© {new Date().getFullYear()} Carleton Computer Science Society -{' '}
<a
className="underline"
href="https://github.com/CarletonComputerScienceSociety/course-graph"
>
GitHub
</a>
</p>
</div>
</footer>
);
}
Loading