Agentic UI: Add a segmented address bar to the site preview - #4366
Merged
Conversation
bcotrim
changed the title
This was referenced Jul 28, 2026
bcotrim
marked this pull request as ready for review
July 28, 2026 12:31
Collaborator
📊 Performance Test ResultsComparing 28cd63e 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) |
nightnei
approved these changes
Jul 30, 2026
nightnei
left a comment
nightnei
left a comment
Contributor
There was a problem hiding this comment.
Everything works as expected 👍
bcotrim
added a commit
that referenced
this pull request
Aug 4, 2026
## Related issues - Fixes STU-2086 ## 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 `canPreview` guard — so files, editor and terminal are reachable without starting the site first. | Running | Stopped | |--------|--------| | <img width="3248" height="2122" alt="image" src="https://github.com/user-attachments/assets/f4cdf989-33ec-4bf5-95fe-8aea44dbb000" /> | <img width="3248" height="2122" alt="image" src="https://github.com/user-attachments/assets/4f9273bc-2aa4-458f-ab07-3dbd478f9004" /> | | <img width="3248" height="2122" alt="image" src="https://github.com/user-attachments/assets/9ee2d477-5b3e-4946-8b26-65fbe558da1e" /> | <img width="3248" height="2122" alt="image" src="https://github.com/user-attachments/assets/a068a9d2-ab38-4749-a42d-f322341bb794" /> | ## Testing Instructions Needs agentic features enabled. 1. Running site: try each destination from the chevron menu. The main half repeats the last one and persists across restarts. 2. Navigate inside the preview, then click the main half on **Browser** — opens that page, not the site root. 3. Stop the site: the control stays, **Browser** and is disabled, the rest still work. 4. Check light + dark, and RTL. ## Pre-merge Checklist - [x] Have you checked for TypeScript, React or other console errors? 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Antonio Sejas <antonio.sejas@automattic.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
3 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.
Related issues
How AI was used in this PR
Ported from the
explore-site-centric-conversation-chromereference branch with Claude Code, trimmed to this PR's scope. Code reviewed and manually QA'd by me.Proposed Changes
Testing Instructions
Agentic UI + a running site.
localStorage.setItem( 'studio:preview-show-database-tab', 'true' )in devtools → Database segment + ⌘3 appear.Pre-merge Checklist
🤖 Generated with Claude Code