Skip to content

feat: show recent people on contact icon - #63147

Open
kristian-zendato wants to merge 1 commit into
masterfrom
feat/people-menu
Open

feat: show recent people on contact icon#63147
kristian-zendato wants to merge 1 commit into
masterfrom
feat/people-menu

Conversation

@kristian-zendato

Copy link
Copy Markdown
Contributor

Summary

Makes the People (contacts) header menu more inviting by showing a stack of up to 3 overlapping avatars of recent contacts on the menu trigger, instead of only the contacts icon.

  • Adds GET /contactsmenu/preview-avatars which returns the first 3 entries from the same unfiltered contacts list as the menu (Manager::getEntries with an empty filter), with optional team filtering matching the existing index endpoint.
  • Frontend loads those contacts on mount and shows an NcAvatar stack when at least 2 are available; otherwise keeps the existing contacts icon.
  • Widens the header trigger (width: fit-content) so the stack does not overlap notifications/profile and remains fully clickable.
  • Adds PHP and Vue unit tests for the new endpoint and trigger behavior.

Screenshot

🏚️ Before

image

🏡 After

image

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@kristian-zendato
kristian-zendato requested review from come-nc, icewind1991, salmart-dev, skjnldsv, sorbaugh and susnux and removed request for a team August 11, 2026 05:08
@kristian-zendato

Copy link
Copy Markdown
Contributor Author

@SebastianKrupinski I created another branch for your comment.
Can you check it again please?

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

otherwise good

Comment on lines +25 to +26
private const PREVIEW_AVATARS_LIMIT = 3;
private const PREVIEW_AVATARS_CACHE_TTL = 300;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
private const PREVIEW_AVATARS_LIMIT = 3;
private const PREVIEW_AVATARS_CACHE_TTL = 300;
private const int PREVIEW_AVATARS_LIMIT = 3;
private const int PREVIEW_AVATARS_CACHE_TTL = 300;

@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team Aug 11, 2026
Signed-off-by: kristian-zendato <kristian.zendato@nextcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement

Projects

Status: 🏗️ At engineering

Development

Successfully merging this pull request may close these issues.

Make People menu more inviting

3 participants