Skip to content

feat(macos): persona avatar in HomeDetailPanel header for assistant rows#31081

Merged
velissa-ai merged 1 commit into
mainfrom
do/detail-panel-persona-avatar
May 19, 2026
Merged

feat(macos): persona avatar in HomeDetailPanel header for assistant rows#31081
velissa-ai merged 1 commit into
mainfrom
do/detail-panel-persona-avatar

Conversation

@velissa-ai
Copy link
Copy Markdown
Collaborator

@velissa-ai velissa-ai commented May 19, 2026

Summary

  • Add a showsPersonaAvatar flag to HomeDetailPanel that swaps the category icon chip for the 32pt persona avatar.
  • Branch on item.fromAssistant == true in PanelCoordinator.homeDetailPanelContent so assistant-initiated rows mirror their list-row treatment from PR feat(home-feed): persona avatar for assistant-sent notifications #31060.
  • Gallery preview updated to exercise both shapes.

Original prompt

Round-2 polish for the home notification surface. PR #31060 swapped the leading icon for the persona avatar on list rows when a FeedItem originated from the assistant (fromAssistant == true), but the right-hand HomeDetailPanel header still rendered a generic bell for the same items. This PR extends the substitution to the detail panel so the surface reads consistently across list and detail views.


Open in Devin Review

…ant-initiated rows

Co-Authored-By: Vellum Assistant <noreply@vellum.ai>
@velissa-ai velissa-ai merged commit 8fc4049 into main May 19, 2026
7 checks passed
@velissa-ai velissa-ai deleted the do/detail-panel-persona-avatar branch May 19, 2026 01:15
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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