Skip to content

fix(routes): set sub-app page titles#1830

Open
DrakeMikels wants to merge 1 commit into
runbox:masterfrom
DrakeMikels:codex/route-page-titles-284
Open

fix(routes): set sub-app page titles#1830
DrakeMikels wants to merge 1 commit into
runbox:masterfrom
DrakeMikels:codex/route-page-titles-284

Conversation

@DrakeMikels
Copy link
Copy Markdown

Summary

  • add Angular route title metadata for the main Runbox sub-app routes
  • cover Mail, Compose, Overview, Welcome, Calendar, Contacts, Account, Help, DKIM, Changelog, Onscreen, Domain Registration, and Login routes
  • use Angular 16 native route titles instead of adding a custom router listener

Closes #284

Verification

  • git diff --check
  • node --check src/app/app.module.ts

Not run

  • npm/ng build or tests: the local system Node/npm install is broken on this machine, and the bundled runtime available here includes node but not npm.

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.

Make page titles reflect the currently viewed sub-app

1 participant