Skip to content

fix(design): k3 - design token audit#228

Open
sifterstudios wants to merge 1 commit intomainfrom
fix/k3-design-token-audit
Open

fix(design): k3 - design token audit#228
sifterstudios wants to merge 1 commit intomainfrom
fix/k3-design-token-audit

Conversation

@sifterstudios
Copy link
Copy Markdown
Owner

Summary

  • Fix Performer ThemeService bug: InitializeAsync was calling _jsRuntime directly on line 56 instead of the local runtime variable (which respected the optional parameter)
  • Fix Performer loading spinner color from hardcoded MudBlazor default purple #594ae2 to var(--mud-palette-primary)
  • Add InfoBrush, WarningBrush, ErrorBrush to Projector App.axaml Light and Dark theme dictionaries (colors existed, brushes were missing)
  • Add ProjectorAxamlSyncTests — 2 facts that parse App.axaml at test time and assert every named <Color> matches DesignTokens — prevents silent drift
  • Add CLAUDE.md with project commands and architecture overview

Notes

  • Projector does not include HighContrast (upstream removed it in k2 — TV display app doesn't need it)
  • Audience and Performer already correctly consumed DesignTokens — no changes needed there

🤖 Generated with Claude Code

- Fix Performer ThemeService using _jsRuntime instead of runtime param (line 56)
- Fix Performer loading spinner using MudBlazor default purple; now uses var(--mud-palette-primary)
- Add InfoBrush, WarningBrush, ErrorBrush to Projector App.axaml Light and Dark themes
- Add ProjectorAxamlSyncTests to catch future drift between App.axaml and DesignTokens
- Resolve merge conflict: Projector dropped HighContrast (TV display app, upstream decision)
- Add CLAUDE.md

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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