Skip to content

Agentic UI: Add the Open in… split button to the preview toolbar - #4365

Open
bcotrim wants to merge 14 commits into
stu-2012-add-preview-site-searchfrom
stu-2086-add-open-in-split-button
Open

Agentic UI: Add the Open in… split button to the preview toolbar#4365
bcotrim wants to merge 14 commits into
stu-2012-add-preview-site-searchfrom
stu-2086-add-open-in-split-button

Conversation

@bcotrim

@bcotrim bcotrim commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

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 canPreview guard — so files, editor and terminal are reachable without starting the site first.

Running Stopped
image image
image image

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

  • Have you checked for TypeScript, React or other console errors?

🤖 Generated with Claude Code

@bcotrim bcotrim changed the title Add the Open in… split button to the preview toolbar Jul 28, 2026
@bcotrim
bcotrim requested review from nightnei and sejas July 28, 2026 10:28
@wpmobilebot

wpmobilebot commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 42381fd vs trunk

app-size

Metric trunk 42381fd Diff Change
App Size (Mac) 1375.36 MB 1375.48 MB +0.13 MB ⚪ 0.0%

site-editor

Metric trunk 42381fd Diff Change
load 1121 ms 1075 ms 46 ms ⚪ 0.0%

site-startup

Metric trunk 42381fd Diff Change
siteCreation 6501 ms 6486 ms 15 ms ⚪ 0.0%
siteStartup 2392 ms 2383 ms 9 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@bcotrim
bcotrim force-pushed the stu-2086-add-open-in-split-button branch from 8e72208 to 42381fd Compare July 28, 2026 11:20
@bcotrim
bcotrim changed the base branch from trunk to stu-2012-add-preview-omnibox July 28, 2026 11:20
@bcotrim
bcotrim force-pushed the stu-2086-add-open-in-split-button branch from 42381fd to 0d3b4d2 Compare July 28, 2026 16:22
@bcotrim
bcotrim changed the base branch from stu-2012-add-preview-omnibox to stu-2012-add-preview-site-search July 28, 2026 16:22
@bcotrim
bcotrim marked this pull request as draft July 28, 2026 16:48
bcotrim added 12 commits July 28, 2026 17:57
…' into stu-2086-add-open-in-split-button

# Conflicts:
#	apps/ui/src/components/site-preview/index.tsx
@bcotrim
bcotrim marked this pull request as ready for review July 31, 2026 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants