Skip to content

Light/dark switch in all three bars - #167

Merged
oblomov-dev merged 1 commit into
mainfrom
claude/beautiful-cerf-sm95qv
Nov 1, 2026
Merged

oblomov-dev merged 1 commit into
mainfrom
claude/beautiful-cerf-sm95qv

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

Follow-up to #166. That PR removed the playground's project bar, and the light/dark switch went with it. It comes back as an icon, in the same place in all three tools.

What changes

  • The switch is a ☾/☀ icon at the right of each bar, beside Full screen: the playground's workbench (#theme), the AI App Builder's bar (#studio-theme) and the AI Agent's (#pilot-theme). The layout editor already has its own ◐.
  • theme.mjs drives every [data-theme-switch] button instead of a single #theme. The glyph shows the theme it switches to. The stored-choice rules are unchanged: a choice equal to the system is forgotten, and a choice made in another tab or restored from the back/forward cache is read again.
  • Phone: the playground's switch moves into the ⋯ menu as "Dark theme".
  • Embed and app-only pages: hidden, as before.
  • AGENTS.md and docs/design/one-site.md updated.

Tests

  • eslint . and typecheck are green.
  • Playwright (chromium): shell, embed, pilot, ai and bfcache specs, 210 tests, all green.
  • Test changes:
    • restored: the theme-switch test, now on the toolbar's button
    • new: a test that the AI App Builder and the AI Agent carry a working switch
    • updated: the phone-menu test expects #theme in the ⋯ menu
    • updated: the embed test expects no switch

Generated by Claude Code

The switch went with the playground's old project bar. It comes back
as an icon at the right of each bar: the playground's workbench, the AI
App Builder's and the AI Agent's, beside Full screen. The layout editor
already carries its own. theme.mjs now drives every [data-theme-switch]
button instead of one #theme. On a phone the playground's switch moves
into the ⋯ menu as "Dark theme". It is hidden in an embed and over an
app on its own.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014YZFmEpRdxrjNxTq4x9J4J
@oblomov-dev
oblomov-dev merged commit 12079af into main Nov 1, 2026
5 of 10 checks passed
@oblomov-dev
oblomov-dev deleted the claude/beautiful-cerf-sm95qv branch November 1, 2026 15:25
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.

2 participants