Skip to content

Show the sidebar Studio wordmark only in the browser UI#4092

Open
shaunandrews wants to merge 1 commit into
trunkfrom
remove-sidebar-studio-wordmark
Open

Show the sidebar Studio wordmark only in the browser UI#4092
shaunandrews wants to merge 1 commit into
trunkfrom
remove-sidebar-studio-wordmark

Conversation

@shaunandrews

@shaunandrews shaunandrews commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

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 appWindow entry 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

  • Desktop: run the app with the agentic UI enabled (ENABLE_AGENTIC_UI=true npm start) — the sidebar header shows no "Studio" label, just the create menu and toggle, centered with the traffic lights.
  • Browser: 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.
  • Check both light and dark color schemes.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors? — npm run typecheck --workspace=apps/ui, eslint, and the site-preview/sidebar-layout tests are clean.

🤖 Generated with Claude Code

@shaunandrews shaunandrews force-pushed the remove-sidebar-studio-wordmark branch from 4892c5b to 54b7291 Compare July 6, 2026 17:31
@shaunandrews shaunandrews changed the title Remove the Studio wordmark from the agentic UI sidebar header Show the sidebar Studio wordmark only in the browser UI Jul 6, 2026
@shaunandrews shaunandrews requested a review from youknowriad July 6, 2026 17:38
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 54b7291 vs trunk

app-size

Metric trunk 54b7291 Diff Change
App Size (Mac) 1411.75 MB 1411.75 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 54b7291 Diff Change
load 1112 ms 1057 ms 55 ms 🟢 -4.9%

site-startup

Metric trunk 54b7291 Diff Change
siteCreation 6488 ms 6494 ms +6 ms ⚪ 0.0%
siteStartup 2386 ms 2391 ms +5 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

@shaunandrews shaunandrews requested a review from bcotrim July 6, 2026 19:25

@youknowriad youknowriad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants