Skip to content

Fix macOS sidebar chrome and rail layout - #29

Merged
wisdomqin merged 1 commit into
mainfrom
agent/fix-macos-sidebar-chrome
Aug 14, 2026
Merged

Fix macOS sidebar chrome and rail layout#29
wisdomqin merged 1 commit into
mainfrom
agent/fix-macos-sidebar-chrome

Conversation

@wisdomqin

Copy link
Copy Markdown
Contributor

What

  • Remove the transient injected macOS titlebar strip and match the native window background to Harness's resolved theme before showing the window.
  • Tighten the DSH logo and original Harness wordmark spacing.
  • Align expanded and collapsed sidebar header spacing with the macOS traffic lights.
  • Use an 80px collapsed rail on macOS while retaining the upstream 56px rail on other platforms.
  • Add regression coverage for initial theme presentation and macOS rail geometry.

Why

The previous first-load titlebar injection produced a black strip in light mode and double-counted vertical sidebar spacing. The default 56px rail was also narrower than the macOS traffic-light group.

User impact

The first rendered frame now matches the selected light or dark theme, sidebar branding keeps stable placement before and after refresh, and the collapsed macOS rail clears the traffic lights while keeping its controls centered.

Checks

  • Fresh npm ci with all patches applied
  • npm test — 41 tests passed
  • npm run typecheck
  • npm run build
  • git diff --check

@wisdomqin
wisdomqin merged commit 7b2b33c into main Aug 14, 2026
4 checks passed
@wisdomqin
wisdomqin deleted the agent/fix-macos-sidebar-chrome branch August 14, 2026 11:41
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