Skip to content

fix(web): make floating browser preview view-only - #526

Merged
incognitojam merged 1 commit into
mainfrom
styal/disable-browser-preview-interaction
Sep 28, 2026
Merged

incognitojam merged 1 commit into
mainfrom
styal/disable-browser-preview-interaction

Conversation

@incognitojam

@incognitojam incognitojam commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

The floating browser preview over chat let clicks, scrolling, and keyboard focus reach the live page, even though the small window is intended for monitoring. That made accidental page changes easy while working in chat.

The floating preview now shields the page from pointer input and makes its hosted webview inert while preserving the live picture. Its move, resize, open-in-panel, and close controls remain usable. Opening the preview in the right panel restores normal page interaction. The fork feature ledger records this behavior.

Validation: focused web tests passed (26 tests), web typecheck passed, the fork feature ledger validated, and CI passed on the PR commit. A paired local web client loaded without page errors. The requester tested the macOS preview build and reported that the floating preview seemed noninteractive. I could not independently capture an Electron interaction test; this change has no intended visual difference to show in static before/after images.


Written by an agent (Codex, GPT-6).

@incognitojam incognitojam added the preview:mac Build an Apple Silicon DMG for this PR on every push. label Sep 28, 2026
@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

macOS preview

The preview download was removed because this PR closed or the preview label was removed.

@incognitojam
incognitojam marked this pull request as ready for review September 28, 2026 19:48
@incognitojam
incognitojam merged commit 8fefaaf into main Sep 28, 2026
27 checks passed
@incognitojam
incognitojam deleted the styal/disable-browser-preview-interaction branch September 28, 2026 19:49
incognitojam pushed a commit that referenced this pull request Sep 29, 2026
Co-authored-by: Claude Code <[email protected]>

Fork adaptation: The browser source keeps fork #197's corner geometry, a 14px frame with the content inset inside the 1px ring and clipped at 13px, and fork #526's view-only shield (preview-surface-continuity), now a shell option only the browser source sets, so floating device streams stay interactive as upstream intends. Agent opens keep the fork's right-panel presentation. The devices guide keeps styal naming.

(cherry picked from commit 8bbe2bf)
Upstream-PR: 11285
incognitojam pushed a commit that referenced this pull request Sep 29, 2026
Co-authored-by: Claude Code <[email protected]>

Fork adaptation: The browser source keeps fork #197's corner geometry, a 14px frame with the content inset inside the 1px ring and clipped at 13px, and fork #526's view-only shield (preview-surface-continuity), now a shell option only the browser source sets, so floating device streams stay interactive as upstream intends. Agent opens keep the fork's right-panel presentation. The devices guide keeps styal naming.

(cherry picked from commit 8bbe2bf)
Upstream-PR: 11285
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview:mac Build an Apple Silicon DMG for this PR on every push. size:S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant