Agentic UI: Add responsive preview modes and the More options menu - #4368
Agentic UI: Add responsive preview modes and the More options menu#4368bcotrim wants to merge 17 commits into
Conversation
📊 Performance Test ResultsComparing 68a2066 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) |
68a2066 to
13ba2a7
Compare
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu # Conflicts: # apps/ui/src/components/site-preview/index.tsx
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
…sponsive-preview-modes-and-the-view-menu
| background-color: var(--wpds-color-bg-surface-neutral-strong, #fff); | ||
| } | ||
|
|
||
| /* The split view's phone pane, beside the desktop frame on the shared |
There was a problem hiding this comment.
I agree the desktop + mobile option doesn't really offer much as it is very hard to see, maybe when we have the full preview panel implemented. Removed for now. cc @shaunandrews
As for the responsive menu, I think it's useful as a developer to be able to confirm the layout easily in different screen sizes.
katinthehatsite
left a comment
There was a problem hiding this comment.
Another observation (which is not a big deal): I noticed that if you select a preview mode, and then navigate to a different site where you select another preview mode, and come back to site A, the site A does not have the preview mode you initially selected and resets to fit pane. I think it can be left as is but just adding it as an observation
…sponsive-preview-modes-and-the-view-menu
…n' into stu-2089-add-responsive-preview-modes-and-the-view-menu # Conflicts: # apps/ui/src/components/site-preview/index.tsx
Following the exploration design, the menu is going to have more options. I think it's fair to keep the 3 dots for now, hovering should show "More options". @shaunandrews do you have any feedback regarding this menu?
|
This is nice, I added this feature to restore the selected mode. |
|
Thanks for the review @katinthehatsite, can you take another look please? 🙇 |
sejas
left a comment
There was a problem hiding this comment.
I think this is very useful feature. I think we could even add multiple views at the same time. So users could see how it looks in desktop AND mobile at the same time.
I tested LTR and RTL and worked as expected.
I miss a way to make the preview pane cover the whole app, kind a maximize the preview.
| const webview = ref.current as WebviewTag | null; | ||
| if ( ! webview ) return; | ||
| appliedViewportRef.current = Boolean( viewport ); | ||
| void applyWebviewViewport( webview, viewport ).catch( () => undefined ); |
There was a problem hiding this comment.
We can consider adding a bounce, in this use effect.
I might make sense to better surface the responsive features, but for now lets stick with the ••• overflow menu to see how people ask for and use these features.
This was in the original design; I think it was decided to remove it above until we add the full screen preview. |




Related issues
How AI was used in this PR
Implemented with Claude Code, ported from the Design Preview exploration (#3975). I reviewed the code and verified manually in the app.
Proposed Changes
Testing Instructions
Full app restart (new main-process IPC handler), agentic UI, a running site.
Pre-merge Checklist
🤖 Generated with Claude Code