Skip to content

fix: keep window draggable while Settings is open (#8) - #10

Merged
msitarzewski merged 1 commit into
mainfrom
fix/issue-8-window-drag-settings-open
May 30, 2026
Merged

msitarzewski merged 1 commit into
mainfrom
fix/issue-8-window-drag-settings-open

Conversation

@msitarzewski

@msitarzewski msitarzewski commented May 30, 2026

Copy link
Copy Markdown
Owner

Fixes #8 — reported by @unluckyquote.

The bug

With Settings open, the dimming scrim (position: fixed; inset: 0; z-index: 90) covered the entire window, including the 36px title bar that serves as the macOS window drag region (data-tauri-drag-region). With the drag region covered, macOS never received the mousedown it uses to begin a window drag — so the window couldn't be moved until Settings was closed.

The fix

Start the scrim below the title bar (inset: 36px 0 0 0) so the drag region stays uncovered and live. This also matches native macOS, where the title bar stays at full opacity while a sheet is open.

Tested on macOS: with Settings open, the window drags by the title bar as expected, and click-outside-to-close still works.

npm run check: 0 errors.

The Settings scrim was `position: fixed; inset: 0`, covering the whole
window including the 36px title bar — which is the macOS window drag
region (`data-tauri-drag-region`). With the drag region covered, macOS
never received the mousedown that starts a window drag, so the window
couldn't be moved until Settings was closed.

Fix: start the scrim below the title bar (`inset: 36px 0 0 0`) so the
drag region stays uncovered and live. This also matches native macOS,
where the title bar stays at full opacity while a sheet is open.

Note: the 36px is coupled to `.titlebar { height: 36px }` in
routes/+page.svelte. The pending title-bar redesign will rework this
area; a shared `--titlebar-height` token would decouple it if this
outlives the redesign.

Reported by @unluckyquote. Fixes #8.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@msitarzewski
msitarzewski merged commit d55f269 into main May 30, 2026
msitarzewski added a commit that referenced this pull request May 31, 2026
The issue-8 fix (window draggable while Settings open, PR #10) is on main
but landed after the v0.5.0 tag, so it's unreleased too -- it belongs in
the same next-version notes. Adds a Bug fixes entry for it and credits
@unluckyquote (Nik) alongside @bytepl in Acknowledgments. Retitled the
file to cover both window fixes.

Reported-by: Nik (@unluckyquote)
Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
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.

App window unmovable with 'Settings' open

1 participant