Agentic UI: Let the app window shrink to 420px - #4444
Open
shaunandrews wants to merge 1 commit into
Open
Conversation
…t renderer Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
shaunandrews
changed the title
shaunandrews
marked this pull request as ready for review
August 4, 2026 19:18
katinthehatsite
left a comment
katinthehatsite
left a comment
Contributor
There was a problem hiding this comment.
I had two main concerns:
- when I resize to the smallest possible window with agentic UI, the toolbar looks all messed up if I close the sidebar and remain within the smallest size (see for example, the buttons to open the browser etc.)
I think that it could be one of those:
-
The preview should auto-close below some width threshold
-
Or the minimum width should account for the preview being open
-
I am also finding the that the sidebar in the min-width is not really informative - for example, two similarly named sites get hard to distinguish e.g. My Swift Site and My WordPress site. In that case, it just shows the first two letters:
But I am also generally wondering how useful this small sized window would be for our customers? What do you think?
Contributor
|
As a sidenote, it seems that the PR testing instructions are not related to the code changes? 😅 |
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.
Related issues
None (split out of the
stu-2162-site-header-actionsexploration)How AI was used in this PR
Claude Code cherry-picked this change out of a larger exploration branch and verified it (lint, typecheck) in isolation.
Proposed Changes
View ▸ Reload / Force Reload / Toggle DevTools used Electron's role-based "focused webContents" target. With a site preview visible, clicking into the preview would shift focus to the guest page, so Reload reloaded the site preview instead of the app — leaving no way to reload the Studio renderer itself from the menu.
These menu items now target the app window's webContents explicitly, and are relabelled "Reload App" / "Force Reload App" to make that explicit. In agentic mode, Reload App drops its ⌘R accelerator so ⌘R still reloads the preview (the renderer's own "Reload preview" shortcut); Force Reload App keeps ⌘⇧R.
Testing Instructions
Pre-merge Checklist