Skip to content

feat: add page titles to each route#12

Merged
PixelKnightDev merged 1 commit into
bsoc-bitbyte:mainfrom
Icsylab:feature/add-page-titles
Jun 9, 2026
Merged

feat: add page titles to each route#12
PixelKnightDev merged 1 commit into
bsoc-bitbyte:mainfrom
Icsylab:feature/add-page-titles

Conversation

@Icsylab

@Icsylab Icsylab commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

What

Added a useEffect in each of the 4 page components (Home, Login, PostRide, PostIntent) that sets document.title on mount so the browser tab shows which page the user is on. PostRide and PostIntent also needed useEffect added to their React import.

Fixes #4

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Proof of work

  • Home page sets document.title = "Splitt"
  • Login page sets document.title = "Sign in · Splitt"
  • PostRide page sets document.title = "Post a ride · Splitt"
  • PostIntent page sets document.title = "Post an intent · Splitt"
  • Each title is set inside a useEffect with an empty dependency array

Checklist

  • I claimed this issue before starting work
  • Branch name follows the convention (feature/, fix/, docs/)
  • Commits follow Conventional Commits format
  • CI is passing (lint, prettier, build)
  • I have self-reviewed my own code
  • Any non-obvious decisions are explained with a comment in the code

@PixelKnightDev

Copy link
Copy Markdown
Collaborator

LGTM

@Icsylab

Icsylab commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Thank you for merging!

@PixelKnightDev PixelKnightDev merged commit 507f7d8 into bsoc-bitbyte:main Jun 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Add page titles to each route

2 participants