Skip to content

Fix double-escaped apostrophe in dashboard user menu#294

Merged
simonhamp merged 2 commits intomainfrom
fix-apostrophe-in-user-menu
Mar 23, 2026
Merged

Fix double-escaped apostrophe in dashboard user menu#294
simonhamp merged 2 commits intomainfrom
fix-apostrophe-in-user-menu

Conversation

@simonhamp
Copy link
Member

@simonhamp simonhamp commented Mar 23, 2026

Summary

  • User names containing apostrophes (e.g. "Timmy D'Hooghe") were displayed as HTML entities (D'Hooghe) in the dashboard sidebar and mobile header profile dropdowns
  • Changed name="{{ }}" to :name="" dynamic binding on both <flux:sidebar.profile> and <flux:profile> components to prevent Blade's {{ }} from escaping the value before the Flux component escapes it again
  • Added a feature test to verify names with apostrophes are not double-escaped

Test plan

  • Verify a user with an apostrophe in their name sees it rendered correctly in the sidebar profile dropdown
  • Verify the same for the mobile header profile dropdown
  • Run php artisan test --filter=DashboardLayoutTest

🤖 Generated with Claude Code

Use dynamic :name binding instead of {{ }} interpolation on Flux
profile components to prevent HTML entities being escaped twice.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp marked this pull request as ready for review March 23, 2026 21:13
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp merged commit 23b38c7 into main Mar 23, 2026
2 checks passed
@simonhamp
Copy link
Member Author

@timmydhooghe

@simonhamp simonhamp deleted the fix-apostrophe-in-user-menu branch March 23, 2026 21:53
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