Skip to content

Set CauseStarter tab title from domain branding - #139

Merged
AdamSpitz merged 2 commits into
devfrom
fix/causestarter-tab-title
Aug 25, 2026
Merged

Set CauseStarter tab title from domain branding#139
AdamSpitz merged 2 commits into
devfrom
fix/causestarter-tab-title

Conversation

@AdamSpitz

Copy link
Copy Markdown
Owner

Summary

CauseStarter uses CauseShell rather than AppShell, so the browser tab stayed at Vite's default ui. Domain chrome now sets document.title from branding, and Vite injects the domain name into index.html so the title is correct before JS loads.

Test plan

  • UI unit tests (App.test.tsx) cover title on AppShell and CauseShell paths
  • Refresh http://localhost:5174/ and confirm the tab says CauseStarter (restart Vite if the HTML plugin was not picked up)

@AdamSpitz AdamSpitz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Review receipt.

Reviewed DomainChrome document.title, Vite index.html title plugin, App tests, and AppShell move. No bugs. One suggestion (DOMAIN_TITLES vs manifests) addressed with a sync comment; Vite cannot import TSX manifests.

Reviewed-with: grok
Reviewed-files: 4
Reviewed-commit: c07a4f1

@AdamSpitz
AdamSpitz merged commit 6fa7b3f into dev Aug 25, 2026
3 checks passed
@AdamSpitz
AdamSpitz deleted the fix/causestarter-tab-title branch August 25, 2026 00:09
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.

1 participant