Skip to content

feat(ui): replace busy pulse animation with blue indicator#172

Open
mvanhorn wants to merge 1 commit intodanshapiro:mainfrom
mvanhorn:osc/160-busy-icons-blue
Open

feat(ui): replace busy pulse animation with blue indicator#172
mvanhorn wants to merge 1 commit intodanshapiro:mainfrom
mvanhorn:osc/160-busy-icons-blue

Conversation

@mvanhorn
Copy link

Summary

  • Busy terminal icons now show blue color (text-blue-500) instead of animate-pulse animation
  • Applies to: pane icons in tab bar, status dots, and overflow indicators
  • Consistent with the existing "creating" state which already uses blue

Fixes #160

Changes

  • src/components/TabItem.tsx: Replaced animate-pulse class with text-blue-500/fill-blue-500 for busy state in StatusDot, PaneIcon, and overflow indicator
  • test/unit/client/components/TabItem.test.tsx: Updated assertions from animate-pulse to text-blue-500
  • test/unit/client/components/TabBar.test.tsx: Updated assertions from animate-pulse to text-blue-500

Test plan

  • npm run typecheck passes
  • All 71 TabItem + TabBar tests pass
  • Manual: Start a coding CLI session, verify busy icons show blue instead of pulsing

This contribution was developed with AI assistance (Claude Code).

Busy terminal icons now show a blue color instead of a pulsing animation,
providing a cleaner, less distracting visual indicator.

Fixes danshapiro#160
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.

Change busy icons from pulsing to blue

1 participant