fix(browser): maximize the browser preview over the whole app window#2440
Merged
Vaibhaav-Tiwari merged 2 commits intoJul 8, 2026
Merged
Conversation
Vaibhaav-Tiwari
approved these changes
Jul 8, 2026
f1uke
pushed a commit
to f1uke/agent-orchestrator
that referenced
this pull request
Jul 10, 2026
…gentWrapper#2440) Cherry-picked from upstream 9cda5d4, conflict-resolved for main-fluke. Adopts AgentWrapper#2440's full-window maximize: when the browser is popped out it now renders as a fixed .browser-popout-overlay portaled to <body> (covering the sidebar + topbar), instead of main-fluke's previous in-column BrowserPanelView inside the 72% terminal ResizablePanel. This escapes the [data-panel] column so the native WebContentsView is no longer clamped and fills the whole window. main-fluke's own SessionView additions are preserved: the fileView (FileDiffView) branch and CenterPane's `directory` prop are kept; only the in-column pop-out branch is dropped in favor of the portal overlay. Behavior change (intended, per AgentWrapper#2440 + user confirmation): popping the browser out now covers the entire app window rather than just the terminal column. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The browser preview's "maximize" button (the pop-out toggle in the toolbar) used to swap the preview into the terminal's ResizablePanel, so it only covered the center column (~72% width) and left the inspector rail, sidebar, and topbar visible. That's not a real maximize.
This renders the maximized preview as a fixed overlay portaled to , so it escapes the shell layout and covers the whole app window — sidebar and topbar included. Because it sits outside any resizable panel, the native WebContentsView is no longer clamped to a single column and fills the entire window. Exit via the same toolbar button (now a minimize icon) to return to the rail.
The terminal now stays mounted behind the overlay instead of being torn down and remounted on restore. Also updated the inspector's now-stale "in the center pane" copy to "maximized" and the pop-out test to match.
