Skip to content

Restore global menu bar chrome to capture metrics - #313

Merged
baseGame merged 1 commit into
masterfrom
ui-1to1-menu-bar
Sep 21, 2026
Merged

baseGame merged 1 commit into
masterfrom
ui-1to1-menu-bar

Conversation

@baseGame

Copy link
Copy Markdown
Collaborator

Summary

  • Verify stacked .menu-bar remains 48px (24px title + 24px menus) vs capture title22.5 + menu23.5 CSS (DPR≈2).
  • Paint title band #f0f0f0 (capture) with white menu row; restore dropdown command rows to 22px / 12px (was 20px / 11px).
  • Mark data-menu-density="capture-1to1"; keep Session/View/Tools/Help (and session-specific) labels.

Test plan

  • Vitest shellChromeDensity, AppLayout.test.ts, full src/styles/
  • Pre-commit quality (prettier, eslint, stylelint, vue-tsc, rust fmt/clippy)
  • Spot-check vs docs/assets/.../home.png, folder-compare.png, text-compare.png (e2e still expects .menu-bar height 48px)

Remaining divergences

  • Window title uses product name OpenDiff (capture shows vendor app name).
  • Lucide brand glyph vs native system-menu icon.
  • Menu contents / enablement depth intentionally out of scope; Options / Tol panels out of scope.

Keep the 48px title+menu stack; paint title band #f0f0f0, white menu row,
and raise dropdown command rows to 22px/12px to match top-level labels.
@baseGame
baseGame merged commit 29a3914 into master Sep 21, 2026
1 check passed
@baseGame
baseGame deleted the ui-1to1-menu-bar branch September 21, 2026 16:47
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