fix(ui): macOS app shows the OpenClaw brand icon twice at the top-left#102497
Merged
Conversation
…ebar The sidebar brand row (logo + wordmark) returned in 8295a12 but the macOS-only titlebar mark from #100648 stayed, so the Mac app showed the claw icon twice. Delete the sidebar-native-brand element and its CSS gate; the sidebar brand row is the single brand surface and the native titlebar strip holds only the traffic lights.
openclaw-barnacle
Bot
added
app: web-ui
Contributor
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: fix(ui): macOS app shows the OpenClaw brand icon twice at the top-left This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
Jul 9, 2026
…ebar (openclaw#102497) The sidebar brand row (logo + wordmark) returned in d04137f but the macOS-only titlebar mark from openclaw#100648 stayed, so the Mac app showed the claw icon twice. Delete the sidebar-native-brand element and its CSS gate; the sidebar brand row is the single brand surface and the native titlebar strip holds only the traffic lights.
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.
What Problem This Solves
Fixes an issue where the macOS app window showed the OpenClaw claw mark twice at the top-left: once in the native titlebar strip next to the traffic lights, and again in the sidebar header right below it.
The titlebar brand mark was added in #100648 when the sidebar brand row was removed, so the mark had a home in the otherwise-empty titlebar strip. Commit 8295a12 ("improve(ui): move shell controls into side rails") later reinstated the sidebar brand row (logo + "OpenClaw" wordmark + search + collapse toggle) without removing the titlebar mark, leaving both visible in the Mac app.
Why This Change Was Made
The sidebar brand row is now the canonical brand surface, so the macOS-only
sidebar-native-brandelement and its CSS gate are deleted outright — no conditional hiding, one fewer platform-special path. The native titlebar strip goes back to holding only the traffic lights, which is standard macOS chrome. The Swift-injected.sidebar-shelltop padding (DashboardWindowController.installNativeChromeScript) is untouched and still clears the traffic-light strip.User Impact
macOS app users see a single OpenClaw mark in the sidebar header instead of a doubled icon. Web builds are unaffected (the removed element was CSS-hidden outside the native macOS window).
Evidence
Rendered via the Control UI e2e mock-gateway harness with the macOS native chrome injection replicated (
openclaw-native-macosclass + titlebar CSS, traffic lights mocked for context), matching howDashboardWindowControllerstyles the real window.ui/src/components/app-sidebar.ts−7,ui/src/styles/layout.css−24), no remainingsidebar-native-brandreferences repo-wide.pnpm check:changed(Blacksmith Testbox): green.