Skip to content

fix: use consistent OpenClaw icon in sidebar settings#13354

Merged
DeJeune merged 1 commit intoCherryHQ:mainfrom
younggglcy:fix/openclaw-sidebar-icon-mismatch
Mar 10, 2026
Merged

fix: use consistent OpenClaw icon in sidebar settings#13354
DeJeune merged 1 commit intoCherryHQ:mainfrom
younggglcy:fix/openclaw-sidebar-icon-mismatch

Conversation

@younggglcy
Copy link
Copy Markdown
Contributor

@younggglcy younggglcy commented Mar 10, 2026

What this PR does

Before this PR:
The OpenClaw icon in Display Settings > Sidebar Settings used a generic Bot icon from lucide-react, which was different from the actual OpenClaw icon shown in the left sidebar.

image

After this PR:
The OpenClaw icon in Sidebar Settings now uses OpenClawSidebarIcon (the same custom SVG icon used in the main sidebar), making both icons consistent.

image

Why we need it and why it was done in this way

The sidebar settings panel should display the same icons as the actual sidebar so users can easily identify which item they are configuring. The fix simply replaces the incorrect Bot icon import with the existing OpenClawSidebarIcon component that is already used in Sidebar.tsx.

The following tradeoffs were made: None — this is a straightforward icon replacement.

The following alternatives were considered: None — using the same icon component is the obvious correct approach.

Breaking changes

None.

Special notes for your reviewer

Single-file change. Only the icon mapping in SidebarIconsManager.tsx was modified.

Checklist

Release note

fix: use consistent OpenClaw icon in Display Settings sidebar configuration

Replace the generic Bot icon (lucide-react) with OpenClawSidebarIcon
in SidebarIconsManager to match the icon used in the main sidebar.

Signed-off-by: younggglcy <[email protected]>
Copy link
Copy Markdown

@MontaEllis8 MontaEllis8 left a comment

Choose a reason for hiding this comment

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

✅ Code Review 通过

图标一致性修复。

建议合并。

Copy link
Copy Markdown
Collaborator

@GeorgeDong32 GeorgeDong32 left a comment

Choose a reason for hiding this comment

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

LGTM! 代码质量良好,变更正确。这是一个简单的 UI 一致性修复:将 OpenClaw 图标从通用的 Bot 图标替换为自定义的 OpenClawSidebarIcon 组件,与主侧边栏保持一致。代码修改干净,无安全风险,可以合并。

Copy link
Copy Markdown

@MontaEllis8 MontaEllis8 left a comment

Choose a reason for hiding this comment

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

✅ Code Review 通过

代码审查通过,建议合并。

@DeJeune DeJeune merged commit c11eaab into CherryHQ:main Mar 10, 2026
12 checks passed
@younggglcy younggglcy deleted the fix/openclaw-sidebar-icon-mismatch branch March 10, 2026 18:22
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.

4 participants