Skip to content

Fix duplicate site name in home page title#25

Merged
plx merged 1 commit intomainfrom
plx/fix-title-duplication
Mar 15, 2026
Merged

Fix duplicate site name in home page title#25
plx merged 1 commit intomainfrom
plx/fix-title-duplication

Conversation

@plx
Copy link
Owner

@plx plx commented Mar 15, 2026

Summary

  • Skip appending | Dispatches suffix when the page title already equals the site name
  • Fixes the home page rendering as "Dispatches | Dispatches" in the browser tab

Test plan

  • Verify home page title is just "Dispatches"
  • Verify other pages still show "Page Title | Dispatches"

🤖 Generated with Claude Code

The home page title rendered as "Dispatches | Dispatches" because
PageLayout unconditionally appended "| SITE.NAME". Now skip the
suffix when the page title already matches the site name.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@claude
Copy link

claude bot commented Mar 15, 2026

Review

This is a clean, minimal fix for the duplicate site name issue.

The change is correct and well-scoped. The ternary on line 17 handles the edge case without touching anything else.

One minor note: the fix is string-equality based, which works perfectly here since the home page title is literally set to SITE.NAME. If a future page happened to use the site name as its title for some other reason, it would also skip the suffix — but that is almost certainly the right behavior anyway, so no action needed.

Test plan looks good. Approve when both checklist items are verified.

@github-actions
Copy link
Contributor

PR Validation ✅

All checks passed! Ready for review.

Checks Performed:

✓ Linting
✓ Spell check (source)
✓ Type checking & Build
✓ Spell check (HTML)
✓ Internal link validation
✓ Artifact upload

This is a complete dry-run of the deployment process, ensuring your changes will deploy successfully when merged.

@plx plx mentioned this pull request Mar 15, 2026
4 tasks
@plx plx merged commit fc16430 into main Mar 15, 2026
3 checks passed
@plx plx deleted the plx/fix-title-duplication branch March 15, 2026 18:57
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