Show the sidebar Studio wordmark only in the browser UI#4092
Open
shaunandrews wants to merge 1 commit into
Open
Show the sidebar Studio wordmark only in the browser UI#4092shaunandrews wants to merge 1 commit into
shaunandrews wants to merge 1 commit into
Conversation
4892c5b to
54b7291
Compare
Collaborator
📊 Performance Test ResultsComparing 54b7291 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) |
youknowriad
approved these changes
Jul 6, 2026
youknowriad
left a comment
Contributor
There was a problem hiding this comment.
I wouldn't mind if this is removed in studio ui as well. That was not an intentional addition, I think a wrong conflict handling.
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
Written with Claude Code, directed and reviewed by @shaunandrews.
Proposed Changes
#3953 added a "Studio" wordmark to the agentic UI's sidebar header on every surface. In the browser (
studio ui/ hosted) it earns its place — a tab has no window chrome naming the app, and the header would otherwise be an empty drag-strip. In the desktop app it's redundant (the dock icon and window already establish the app) and crowds the header as the sidebar narrows.This scopes the wordmark to where it helps: it now shows only when the UI is not running in a dedicated app window. The signal is a new
appWindowentry in the connector capabilities layer #3953 introduced for exactly this kind of host difference — true for the Electron connector, false for the local and hosted ones — so the distinction stays declarative and mockable rather than sniffed from the environment.User impact: the desktop sidebar header returns to just the create menu and sidebar toggle; the browser surface keeps its wordmark unchanged.
Testing Instructions
ENABLE_AGENTIC_UI=true npm start) — the sidebar header shows no "Studio" label, just the create menu and toggle, centered with the traffic lights.npm run cli:build && node apps/cli/dist/cli/main.mjs ui— the header still shows the "Studio" wordmark, aligned with the site list below.Pre-merge Checklist
npm run typecheck --workspace=apps/ui, eslint, and the site-preview/sidebar-layout tests are clean.🤖 Generated with Claude Code