Skip to content

fix(ui): restore contrast to editor controls - #266

Merged
DavidBabinec merged 1 commit into
mainfrom
fix/canvas-mode-toggle-background
Jul 24, 2026
Merged

fix(ui): restore contrast to editor controls#266
DavidBabinec merged 1 commit into
mainfrom
fix/canvas-mode-toggle-background

Conversation

@DavidBabinec

Copy link
Copy Markdown
Contributor

What changed

  • match the canvas mode toggle shell to the canvas/body background
  • restore the shared Switch primitive's high-contrast checked, unchecked, and disabled states
  • add regression coverage for both token choices

Why

The canvas mode toggle was painted with a raised surface color instead of the black canvas chrome. Separately, a token migration had collapsed the Switch checked track onto a near-identical surface tone, making its state difficult to read on editor cards.

Impact

Canvas chrome now blends into the intended background, while switches remain clearly distinguishable across dark and light editor surfaces.

Verification

  • bun test src/__tests__/ui/neutralInteractiveChrome.test.ts src/__tests__/canvas/canvasNotch.test.ts src/__tests__/canvas/canvasMode.test.tsx (29 passing)
  • bun run lint
  • bun run build
  • disposable local browser pass covering both checked and unchecked switch states

@DavidBabinec
DavidBabinec marked this pull request as ready for review July 24, 2026 19:01
@DavidBabinec
DavidBabinec merged commit cb4fa64 into main Jul 24, 2026
9 checks passed
@DavidBabinec
DavidBabinec deleted the fix/canvas-mode-toggle-background branch July 24, 2026 19:01
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