Skip to content

fix(app): update page title per sub-app#1837

Open
dttxorg wants to merge 3 commits into
runbox:masterfrom
dttxorg:fix/page-title-subapp
Open

fix(app): update page title per sub-app#1837
dttxorg wants to merge 3 commits into
runbox:masterfrom
dttxorg:fix/page-title-subapp

Conversation

@dttxorg
Copy link
Copy Markdown

@dttxorg dttxorg commented May 13, 2026

Summary

  • add a page title service that derives the Runbox sub-app name from the current route
  • wire the service into app startup so the document title updates after navigation
  • add unit coverage for mail/default routes, sub-app routes, and NavigationEnd updates

Fixes #284

Testing

  • npm run lint -- --lint-file-patterns src/app/page-title.service.ts --lint-file-patterns src/app/page-title.service.spec.ts --lint-file-patterns src/app/app.module.ts
  • npm run build
  • npm test -- --watch=false --include src/app/page-title.service.spec.ts (bundle generation succeeds, but Karma cannot launch because Firefox is not installed in this local environment: "No binary for Firefox browser on your platform. Please, set FIREFOX_BIN env variable.")

AI disclosure

This contribution was implemented with assistance from OpenAI Codex in src/app/page-title.service.ts, src/app/page-title.service.spec.ts, and src/app/app.module.ts.

dttxorg added 2 commits May 13, 2026 13:21
Set the document title from the active route so browser tabs identify the current Runbox sub-app.

Fixes runbox#284
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