Skip to content

macos: add participants overflow count badge in toolbar#95

Open
shindgew wants to merge 2 commits into
inline-chat:mainfrom
shindgew:feature/94-toolbar-participants-overflow-badge
Open

macos: add participants overflow count badge in toolbar#95
shindgew wants to merge 2 commits into
inline-chat:mainfrom
shindgew:feature/94-toolbar-participants-overflow-badge

Conversation

@shindgew
Copy link
Copy Markdown

@shindgew shindgew commented May 28, 2026

Implements #94.

Adds a +N overflow badge to the participants avatar stack in the chat toolbar for threads that have more than 3 participants.

Changes

  • ParticipantAvatarStack now renders only the first 3 avatars + a circular accent badge showing the overflow count (using compact notation).
  • Updated ParticipantsButton width calculation to reserve space for the badge.
  • The stack is now public and handles its own truncation + badge logic.

Details

  • Matches the existing visual language of avatar stacks (overlap, shadows, borders) used elsewhere in the app (e.g. reactions).
  • The original overflowCount placeholder in the file is now used.
  • Clean, scoped change: only two files, no behavior changes for ≤3 participants.

Testing

  • Local verification on macOS for participant counts 0–3 (no badge) and 4+ (badge appears, width adjusts correctly).
  • No impact on 1:1 chats (the button is only shown for threads).

Risks / Readiness

  • Purely presentational / layout tweak in the toolbar.
  • No security, performance, or data changes.
  • Production ready (low risk, easy to iterate on badge styling later if needed — e.g. font size for very large numbers).

shindgew added 2 commits May 28, 2026 12:56
- Limit visible participants to 3
- Show +N indicator for overflow participants
- Adjust zIndex ordering for proper stacking
- Update button width calculation to account for overflow indicator
- Remove unnecessary horizontal padding from stack
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