Skip to content

Fix macOS top chrome drag rail / 修复 macOS 顶部窗口拖拽区域#3891

Merged
SivanCola merged 1 commit into
esengine:main-v2from
SivanCola:fix/macos-top-drag-rail
Jun 11, 2026
Merged

Fix macOS top chrome drag rail / 修复 macOS 顶部窗口拖拽区域#3891
SivanCola merged 1 commit into
esengine:main-v2from
SivanCola:fix/macos-top-drag-rail

Conversation

@SivanCola

Copy link
Copy Markdown
Collaborator

Summary

  • Add a macOS-only invisible top drag rail above the tab strip so the window remains draggable even when tabs fill the chrome.
  • Keep tab controls and buttons as no-drag, while the rail exposes a subtle full-width hover indicator and accent-colored active state.
  • Leave Windows and Linux behavior unchanged.

Root Cause

Reasonix uses an inset/hidden macOS title bar, so the webview must explicitly provide draggable regions. The tab strip is interactive and opts out of Wails dragging, which can leave no reliable drag target when the top chrome is full of tabs.

Validation

  • pnpm --dir desktop/frontend run build
  • Browser preview with platform=darwin verified that the rail is the hit target, the rail is draggable, tab controls remain no-drag, and hover/active indicators render at the same full width.

Related: #3843, #3852, #3853

@SivanCola SivanCola requested a review from esengine as a code owner June 10, 2026 19:24
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@esengine esengine left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice — a dedicated invisible drag rail scoped to .app--darwin .app-chrome--tabs is the clean way to restore a draggable target when the tab strip eats the chrome, and leaving the tab controls as no-drag keeps them clickable. Windows/Linux untouched. Thanks!

@SivanCola SivanCola merged commit 3648d74 into esengine:main-v2 Jun 11, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**) v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants