Skip to content

fix(web-ui): use real chat app brand icons - #2621

Merged
bobleer merged 2 commits into
GCWing:1.0.0-explorefrom
bobleer:bob/real-chat-app-icons
Aug 29, 2026
Merged

fix(web-ui): use real chat app brand icons#2621
bobleer merged 2 commits into
GCWing:1.0.0-explorefrom
bobleer:bob/real-chat-app-icons

Conversation

@bobleer

@bobleer bobleer commented Aug 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • replace the generic Lucide placeholders with recognizable monochrome Telegram, Feishu, and WeChat SVG marks
  • preserve the existing identity badge sizing and currentColor theme behavior
  • repair upstream frontend regressions found by CI: preserve feature-owned Appearance attributes through ScrollArea, complete the Hooks icon mock, and align Design System registry/Design Lab contract expectations
  • add regression coverage for all three brand icons and ScrollArea Appearance forwarding

Testing

  • pnpm --dir src/web-ui run test:run (Node 22: 574 files, 4148 tests passed)
  • pnpm run check:web
  • pnpm run design-system:check

Remote scenarios

The chat-icon change was exercised in the shared Web UI Remote Connect surface. No remote-workspace, remote-control, Peer Device Mode, or Detached Dispatch protocol/runtime behavior changed; the remaining fixes affect shared frontend tests and presentation contracts only.

@bobleer
bobleer merged commit f83df11 into GCWing:1.0.0-explore Aug 29, 2026
8 checks passed
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.

1 participant