Skip to content

Restore the bar's look and layout under the theme provider - #3

Merged
SaulDoesCode merged 1 commit into
mainfrom
claude/practical-wozniak-y9xdb4
Oct 27, 2026
Merged

SaulDoesCode merged 1 commit into
mainfrom
claude/practical-wozniak-y9xdb4

Conversation

@SaulDoesCode

Copy link
Copy Markdown
Owner

Fixes styling regressions from #1.

Cause

#1 added zgui's ThemeProvider for light mode, which defines the design tokens on :root. Before that, the bar's Input read tokens that didn't exist, so it had no border, no focus ring, and inherited its font size and colour from .inp. Once the tokens were defined, the bar picked up:

  • a 1px border
  • a 3px focus ring
  • the token font size (smaller than the bar's 20px)

#1 also wrapped the app in a .app column sized 100vw × 100vh, which changed how the page laid out and centred.

Changes

  • Neutralise the .zui-input border, ring, radius, background and font inside .inp and .term-menu. The selector is more specific than the component's own, so the path field in the escape menu keeps its themed look.
  • .app is now display: contents, so the bar, sums and entries lay out against the root exactly as before. The window background comes from the theme's background token on :root: pure black in dark mode, as before, and cream in light mode.
  • The hint/status row only renders when it has text, so the gap between the bar and the sums is back to the original spacing.
  • The alt-name box is back to max-width: 256px.
  • zgui is updated to the latest commit. It now needs Rust 1.90, so rust-version and the README are bumped to match.

Checks

  • cargo clippy --all-targets reports no warnings and all 33 tests pass.
  • I couldn't take screenshots: the latest zgui rejects the software GPU adapter in this headless environment, so this still needs a look on a real desktop.

Generated by Claude Code

The ThemeProvider added for light mode switched on zgui's field tokens, so
the bar's Input grew a border, a focus ring and the token font size, and the
full-viewport .app wrapper changed how the page laid out.

- neutralise the zui-input chrome inside the bar and the alt name box
- make .app display: contents so the layout is the original root layout;
  the window background comes from the theme's background token on :root
  (pure black in dark, cream in light)
- only render the hint row when it has something to say
- alt name box back to 256px
- zgui updated to the latest commit (needs Rust 1.90)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Q9xKZd6m8irSsnZNnbkucs
@SaulDoesCode
SaulDoesCode marked this pull request as ready for review October 27, 2026 12:46
@SaulDoesCode
SaulDoesCode merged commit 45f34af into main Oct 27, 2026
0 of 3 checks passed
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