Agentic UI: Add the Open in… split button to the preview toolbar - #4365
Open
bcotrim wants to merge 14 commits into
Open
Agentic UI: Add the Open in… split button to the preview toolbar#4365bcotrim wants to merge 14 commits into
bcotrim wants to merge 14 commits into
Conversation
Collaborator
📊 Performance Test ResultsComparing 42381fd vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
bcotrim
force-pushed
the
stu-2086-add-open-in-split-button
branch
from
July 28, 2026 11:20
8e72208 to
42381fd
Compare
1 task
bcotrim
force-pushed
the
stu-2086-add-open-in-split-button
branch
from
July 28, 2026 16:22
42381fd to
0d3b4d2
Compare
bcotrim
changed the base branch from
stu-2012-add-preview-omnibox
to
stu-2012-add-preview-site-search
July 28, 2026 16:22
bcotrim
marked this pull request as draft
July 28, 2026 16:48
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button # Conflicts: # apps/ui/src/components/site-preview/index.tsx
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
…' into stu-2086-add-open-in-split-button
bcotrim
marked this pull request as ready for review
July 31, 2026 12:25
1 task
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
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.
Related issues
How AI was used in this PR
Claude Code ported
open-in-menu/from the Design Preview exploration (45fd6fb4c, #3975) and wrote the tests. I reviewed the diff and tested in the app.Proposed Changes
The preview toolbar's external-link icon becomes an Open in… split button: the left half repeats the last destination (persisted), the chevron opens Browser, Finder/File Explorer, editor, terminal, phpMyAdmin. Browser opens the page currently shown in the preview, and is the only destination disabled while the site is stopped.
The control itself stays in the toolbar in every state, including stopped — unlike the button it replaces, which #4366 moved inside the
canPreviewguard — so files, editor and terminal are reachable without starting the site first.Testing Instructions
Needs agentic features enabled.
Pre-merge Checklist
🤖 Generated with Claude Code