Skip to content

style: unify the ink accent, fix text contrast, and clarify disabled states - #82

Open
jibraaan wants to merge 1 commit into
CopilotKit:mainfrom
jibraaan:pr/ui-polish
Open

jibraaan wants to merge 1 commit into
CopilotKit:mainfrom
jibraaan:pr/ui-polish

Conversation

@jibraaan

@jibraaan jibraaan commented Oct 4, 2026

Copy link
Copy Markdown

What and why

The compact-chrome layer moved the app to an ink (near-black) palette, but several older surfaces kept the original lavender: .primary buttons, focus rings, checkboxes and the setup notes. As a result, Settings looked like a different app from the chat screen. A few states were also hard to read or misleading.

  • One accent: primary buttons, focus rings and checkboxes use the ink accent.
  • Disabled controls look disabled: before setup, the send button was near-black at 90% opacity and looked clickable. Disabled primary and send buttons are now neutral gray.
  • Contrast: secondary text in the "Connect your Dot" card, the config notes and the permission descriptions measured about 2.4–2.7:1. It now measures 5.3:1 (WCAG AA).
  • Settings: missing environment variables are shown as code.
  • Sidebar: Spaces gets the same spacing as Dots (it sat flush against the Dot row), nav labels stay on one line, and zero-count badges are hidden.

This is a refinement pass, not a redesign: no copy, layout or behavior changes beyond the above. The CSS is a single appended, clearly labeled layer, following the existing pattern of appended layers. JSX changes are limited to the badge condition and the env-var code formatting.

Verification

  • npm run check-format, lint, typecheck, test (164 passing) and build all pass.
  • Checked in the browser at desktop and 375px widths: no horizontal overflow, and contrast measured as above.
  • Not covered: views that need configured services (active chat, call) weren't visually reviewed.

Independent of the dark-mode PR; either can merge first.

🤖 Generated with Claude Code

The compact-chrome layer moved the app to an ink palette, but primary
buttons, focus rings, checkboxes, and setup notes kept the original
lavender, so Settings and Contacts looked like a different app.

- Primary buttons, focus rings, and checkboxes use one ink accent.
- Disabled primary/send/connect buttons look disabled (the send button
  was near-black at 90% opacity before setup).
- Secondary text in setup cards, config notes, and permission rows meets
  WCAG AA (was ~2.4-2.7:1, now 5.3:1).
- Settings shows missing environment variables as code.
- Sidebar: Spaces gets the same spacing as Dots, labels stay on one line,
  and zero-count badges are hidden.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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