Skip to content

cmux135: #12540 cloud leading icon - #12543

Merged
austinywang merged 2 commits into
mainfrom
issue-12540-cloud-leading-icon
Sep 13, 2026
Merged

austinywang merged 2 commits into
mainfrom
issue-12540-cloud-leading-icon

Conversation

@austinywang

@austinywang austinywang commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

Cloud workspace rows now put the cloud outline immediately before the workspace name in both sidebar renderers. Pinned Cloud workspaces keep the existing pin followed by the cloud identity; the old trailing badge is removed.

The badge retains its existing color, tooltip, accessibility label, font scaling, and fixed size. Title truncation and remote directory provenance are unchanged. Leading identity consumes the same total title-row width as the previous trailing accessory.

Validation:

  • Test-only commit 944d9107e8 changes the real AppKit geometry assertions before the fix in 3c0933d657.
  • Focused tests cover 64 combinations of width, pinning, selection, density, appearance, and magnification, with isolated screenshot attachments and /home/cmux directory checks.
  • Swift file-length guard passes with zero production-file growth; localization audit passes all 6 catalogs and 9 required locales. No user-facing strings changed.
  • Canonical Codex autoreview and Aziz policy checks are clean; merge-conflict gate passes against origin/main.
  • Build/test blocker: the tagged fleet build and focused test run stop on pre-existing duplicate CmuxTuiSurfaceProvider declarations. Runtime tests and screenshot evidence have not executed yet. An existing-owner dependency is being investigated.
  • The default development backend hostname is unavailable; the isolated layout build uses supported CMUX_DEV_BACKEND_MODE=off. No production backend changes or local app launches.

Fixes #12540
Related: #12291, #12347

@vercel

vercel Bot commented Sep 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
cmux166 Building Building Preview Sep 13, 2026 11:40pm UTC
cmux41 Building Building Preview Sep 13, 2026 11:40pm UTC

@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 4 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: d49fc00f-54a9-4be4-80e8-34fddf20c95c

📥 Commits

Reviewing files that changed from the base of the PR and between 136dcd3 and 3c0933d.

📒 Files selected for processing (4)
  • Sources/ContentView.swift
  • Sources/NSImageView+SidebarWorkspaceAccessory.swift
  • Sources/Sidebar/AppKitList/Cells/SidebarWorkspaceRowCellView.swift
  • cmuxTests/SidebarCloudWorkspaceBadgeTests.swift

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@austinywang
austinywang merged commit 8b6c6e0 into main Sep 13, 2026
18 of 23 checks passed
rustybret pushed a commit to rustybret/bmux that referenced this pull request Sep 14, 2026
22ae1c5 Merge pull request manaflow-ai#12550 from manaflow-ai/issue-12547-nightly-provider-duplicates
528d92f fix: deduplicate Cloud provider implementations
8b6c6e0 Merge pull request manaflow-ai#12543 from manaflow-ai/issue-12540-cloud-leading-icon
3c0933d fix: place Cloud workspace identity before sidebar titles
944d910 test: require Cloud workspace badge before sidebar title
136dcd3 Merge pull request manaflow-ai#12534 from manaflow-ai/issue-12533-vercel-staging-guest-assets
cc45e9d test(web): pin guest prompt asset bytes
5f1d135 fix(web): load guest prompt assets in Vercel builds
1f17bb3 Merge pull request manaflow-ai#4025 from manaflow-ai/issue-1756-pane-resize-keybindings
8831b43 Merge pull request manaflow-ai#12519 from manaflow-ai/issue-4701-webkit-activity-crash
2acbf93 Merge pull request manaflow-ai#12511 from manaflow-ai/issue-12480-cloud-pane-modal
bc6d2e0 Merge origin/main into issue-4701-webkit-activity-crash
59facc1 Merge remote-tracking branch 'origin/main' into issue-1756-pane-resize-keybindings
2dd797c Merge remote-tracking branch 'origin/main' into issue-1756-pane-resize-keybindings
0f66315 test: cover revealed browser history restoration
3b4b441 test: import browser viewport package
0408681 fix: keep callback validation inside browser owner
4d742d4 fix: expose browser callback generation check
dff60ee Merge remote-tracking branch 'origin/main' into issue-4701-webkit-activity-crash
f53518a fix: fence stale browser lifecycle callbacks
e053fb7 fix: complete cross-file browser lifecycle access
21d4bd9 fix: expose panel state to web content lifecycle owner
d3c3cb1 fix: include browser stream state in discard snapshot
0e576a7 fix: keep browser delegate accessible to recovery lifecycle
57897e8 fix: expose portal lock for browser lifecycle extension
8968961 fix: defer WebKit WebContent replacement until recovery
8d41c77 test: defer WebKit termination replacement until recovery
4bc58c7 fix: fence superseded cloud pane failures
8c35c0a test: exercise live Dock tab identity in link routing fixture
1c001e1 fix: isolate cloud failure ownership on main actor
14de512 test: model cloud tab identity in failure fixture
a5df8d0 test: assert cloud failure copy is sanitized
8d9f0a3 test: supply drag registry in Cloud sidebar scale fixture
e3566cd fix: harden cloud pane failure state
74d0d40 test: align terminal link fixture with current container protocol
2a80d3f Merge remote-tracking branch 'origin/main' into issue-12480-cloud-pane-modal
8787859 fix: show cloud pane creation failures inline
b75b987 fix: use public Bonsplit tab identity for Cloud layout projection
9713e46 test: preserve settings isolation when resize actions are absent
8de947b fix: restore terminal primitives lost in Cloud provider extraction
26cc800 fix: accept indexed resources in Cloud workspace reconciliation
c7954be fix: keep resize settings typed and expose Dock palette actions
8154cfb fix: quote resize test extension path in Xcode project
d077aec test: disambiguate app shortcut type in resize coverage
1b6d97a Merge remote-tracking branch 'origin/main' into issue-1756-pane-resize-keybindings
f1cb666 style: trim extracted settings file boundaries
2b4b176 feat: finish configurable pane resize shortcuts across workspace and Dock
253bbbe Merge remote-tracking branch 'origin/issue-1756-pane-resize-keybindings' into issue-1756-pane-resize-keybindings
766c819 test: cover pane resize routing, repeat, and settings validation
9548f46 test: cover non-modal cloud pane creation failure
3eabdcc fix: address pane resize review feedback
7f87e23 Merge remote-tracking branch 'origin/main' into issue-1756-pane-resize-keybindings
4617dac feat: add pane resize shortcuts

This branch was successfully deployed

2 active deployments
Preview – cmux166 — 3c0933d6 Deployed Sep 14, 2026 by vercel[bot]
Preview – cmux41 — 3c0933d6 Deployed Sep 14, 2026 by vercel[bot]
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.

Move Cloud workspace icons to the leading position beside workspace names like the pin

1 participant