Repository navigation
menu bar: c11 glyph, drop integrations submenu, surface-aware tooltip - #106
Merged
Merged
Conversation
…, surface-aware tooltip Three changes scoped to MenuBarExtraController and MenuBarIconRenderer. 1. Replace the menu bar glyph. The status-item icon was rendering the canonical cmux right-pointing chevron from `Icon Center Image Artwork.svg`, a brand conflict for c11 users. Swap it for an "Open-Center Plus": a four-armed cardinal cross with a centered square cutout, derived from the c11 app icon's four-fold tile motif. Retune the glyph rect from `(1.2, 1.5, 11.6, 15.0)` to a square `(1.0, 1.0, 11.0, 11.0)` so the symmetric plus reads square. Stays a single fill path with template-image recoloring; badge draw on top is unchanged. 2. Remove the Integrations submenu. The "Install Claude Code / Codex / OpenCode / Kimi" submenu pointed at `c11 install <tui>`, a direction the project doc explicitly rejects. Pure delete of `integrationsItem`, `integrationsSubmenu`, `rebuildIntegrationsSubmenu`, `currentInstalledIntegrations`, `installIntegrationAction`, and the `openIntegrationInstallSurface` AppDelegate helper that only the submenu called. 3. Enrich the status-item hover tooltip. With unread notifications, append the deduplicated surface titles below the count on a second line: "c11: 3 unread notifications\\nTitle A · Title B · Title C". Capped to maxInlineNotificationItems entries with a trailing ellipsis when there are more. Three `menu.integrations*` entries in `Resources/Localizable.xcstrings` are now orphaned. Cleanup left for a follow-up sweep alongside other xcstrings maintenance to avoid co-mingling with About-panel work already in flight on that file. Build: tag `menubar-glyph` succeeds; running instance verified to load and draw the new glyph. Visual verification of the menu (Integrations gone) and tooltip (surface titles on second line) is the operator's test plan. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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
1 participant
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.
Summary
Three scoped changes to the macOS menu bar extra (
MenuBarExtraController+MenuBarIconRenderer):(1.2, 1.5, 11.6, 15.0)to a square(1.0, 1.0, 11.0, 11.0)so the symmetric plus reads square.c11 install <tui>, a direction the projectCLAUDE.mdexplicitly rejects. Removes the menu items, the rebuild + filesystem-marker scan helpers, and theopenIntegrationInstallSurfaceAppDelegate helper that only the submenu called.c11: 3 unread notifications\nTitle A · Title B · Title C. Capped tomaxInlineNotificationItems; trailing ellipsis if more.Why these three together
drawGlyphliterally hardcoded the cmux center-mark path).c11skill (set-status/ surface manifest) so the hover signal becomes self-documenting at zero menu real-estate cost.What stays unchanged (deliberate, per upstream-compatibility lean for cmux migrants): badge, inline last-6 notification list, Show Notifications, Jump to Latest Unread, Mark All Read, Clear All, "No unread" state hint, Check for Updates, Preferences, Quit.
Follow-ups
menu.integrations*entries inResources/Localizable.xcstringsare now orphaned. Left for a follow-up sweep so this PR doesn't co-mingle with About-panel xcstrings work already in flight on that file.Test plan
./scripts/reload.sh --tag menubar-glyph→ succeeds. (Verified locally.)c11.·, capped at six with…if more.🤖 Generated with Claude Code