feat: show recent people on contact icon

Signed-off-by: Kristian Zendato <kristian.zendato@nextcloud.com>

Author: kristian-zendato

core/Controller/ContactsMenuController.php

@@ -70,4 +70,31 @@ public function findOne(int $shareType, string $shareWith) {
 	public function getTeams(): array {
 		return $this->teamManager->getTeamsForUser($this->userSession->getUser()->getUID());
 	}
+
+	/**
+	 * Top contacts for the People menu header avatar stack (max 3).
+	 * Same source/order as the contacts menu with an empty filter.
+	 *
+	 * @return list<IEntry>
+	 * @throws Exception
+	 */
+	#[NoAdminRequired]
+	#[FrontpageRoute(verb: 'GET', url: '/contactsmenu/preview-avatars')]
+	public function previewAvatars(?string $teamId = null): array {
+		$user = $this->userSession->getUser();
+		if ($user === null) {
+			return [];
+		}
+
+		$entries = $this->manager->getEntries($user, '');
+		if ($teamId !== null && $teamId !== '') {
+			$memberIds = $this->teamManager->getMembersOfTeam($teamId, $user->getUID());
+			$entries['contacts'] = array_filter(
+				$entries['contacts'],
+				fn (IEntry $entry) => array_key_exists($entry->getProperty('UID'), $memberIds)
+			);
+		}
+
+		return array_values(array_slice($entries['contacts'], 0, 3));
+	}
 }

core/src/tests/views/ContactsMenu.spec.ts

@@ -19,6 +19,20 @@ vi.mock('@nextcloud/auth', () => ({
 
 afterEach(cleanup)
 
+function mockDefaultGets(previewUsers: Array<{ uid: string, fullName: string, isUser?: boolean }> = []) {
+	axios.get.mockImplementation(async (url: string) => {
+		if (String(url).includes('/contactsmenu/preview-avatars')) {
+			return {
+				data: previewUsers.map((user) => ({
+					isUser: true,
+					...user,
+				})),
+			}
+		}
+		return { data: [] }
+	})
+}
+
 describe('ContactsMenu', function() {
 	it('shows a loading text', async () => {
 		const { promise, resolve } = Promise.withResolvers<void>()
@@ -124,4 +138,38 @@ describe('ContactsMenu', function() {
 		expect(items[0]!.textContent).toContain('Acosta Lancaster')
 		expect(items[1]!.textContent).toContain('Adeline Snider')
 	})
+
+	it('shows the contacts icon when fewer than two preview users are available', async () => {
+		mockDefaultGets([{ uid: 'alice', fullName: 'Alice', isUser: true }])
+		axios.post.mockResolvedValue({
+			data: { contacts: [], contactsAppEnabled: false },
+		})
+
+		const view = render(ContactsMenu)
+		await view.findByRole('button')
+
+		expect(view.container.querySelector('.contactsmenu__trigger-avatars')).toBeNull()
+		expect(view.container.querySelector('.contactsmenu__trigger-icon')).toBeTruthy()
+	})
+
+	it('shows an avatar stack when at least two preview users are available', async () => {
+		mockDefaultGets([
+			{ uid: 'alice', fullName: 'Alice', isUser: true },
+			{ uid: 'contact-1', fullName: 'External Contact', isUser: false },
+			{ uid: 'bob', fullName: 'Bob', isUser: true },
+		])
+		axios.post.mockResolvedValue({
+			data: { contacts: [], contactsAppEnabled: false },
+		})
+
+		const view = render(ContactsMenu)
+		await view.findByRole('button')
+
+		// wait for onMounted preview load
+		await vi.waitFor(() => {
+			expect(view.container.querySelector('.contactsmenu__trigger-avatars')).toBeTruthy()
+		})
+		expect(view.container.querySelectorAll('.contactsmenu__trigger-avatars__avatar')).toHaveLength(3)
+		expect(view.container.querySelector('.contactsmenu__trigger-icon')).toBeNull()
+	})
 })

core/src/views/ContactsMenu.vue

@@ -14,6 +14,7 @@ import debounce from 'debounce'
 import { computed, nextTick, onMounted, ref, watch } from 'vue'
 import NcActionButton from '@nextcloud/vue/components/NcActionButton'
 import NcActions from '@nextcloud/vue/components/NcActions'
+import NcAvatar from '@nextcloud/vue/components/NcAvatar'
 import NcButton from '@nextcloud/vue/components/NcButton'
 import NcEmptyContent from '@nextcloud/vue/components/NcEmptyContent'
 import NcHeaderMenu from '@nextcloud/vue/components/NcHeaderMenu'
@@ -23,6 +24,12 @@ import NcTextField from '@nextcloud/vue/components/NcTextField'
 import ContactMenuEntry from '../components/ContactsMenu/ContactMenuEntry.vue'
 import logger from '../logger.js'
 
+interface IPreviewUser {
+	uid: string
+	fullName: string
+	isUser: boolean
+}
+
 const storage = getBuilder('core:contacts')
 	.persist(true)
 	.clearOnLogout(true)
@@ -44,6 +51,8 @@ const searchTerm = ref('')
 const teams = ref<ITeam[]>([])
 const selectedTeam = ref<string>('$_all_$')
 const selectedTeamName = computed(() => teams.value.find((t) => t.teamId === selectedTeam.value)?.displayName)
+const previewUsers = ref<IPreviewUser[]>([])
+const showAvatarStack = computed(() => previewUsers.value.length >= 2)
 
 onMounted(async () => {
 	const team = storage.getItem('core:contacts:team')
@@ -60,13 +69,32 @@ onMounted(async () => {
 		}
 	}
 	teams.value = [...userTeams]
+	await loadPreviewAvatars()
 })
 
 watch(selectedTeam, () => {
 	storage.setItem('core:contacts:team', JSON.stringify(selectedTeam.value))
 	getContacts(searchTerm.value)
+	loadPreviewAvatars()
 })
 
+/**
+ * Load avatars for the People menu header trigger
+ */
+async function loadPreviewAvatars() {
+	try {
+		const { data } = await axios.get<IPreviewUser[]>(generateUrl('/contactsmenu/preview-avatars'), {
+			params: {
+				teamId: selectedTeam.value !== '$_all_$' ? selectedTeam.value : undefined,
+			},
+		})
+		previewUsers.value = data
+	} catch (error) {
+		logger.error('could not load preview avatars', { error })
+		previewUsers.value = []
+	}
+}
+
 /**
  * Load contacts when opening the menu
  */
@@ -145,11 +173,32 @@ const userTeams: ITeam[] = []
 	<NcHeaderMenu
 		id="contactsmenu"
 		class="contactsmenu"
+		:class="{ 'contactsmenu--avatar-stack': showAvatarStack }"
 		:aria-label="t('core', 'Search contacts')"
 		exclude-click-outside-selectors=".v-popper__popper"
 		@open="onOpened">
 		<template #trigger>
-			<NcIconSvgWrapper class="contactsmenu__trigger-icon" :path="mdiContacts" />
+			<span
+				v-if="showAvatarStack"
+				class="contactsmenu__trigger-avatars"
+				aria-hidden="true">
+				<NcAvatar
+					v-for="(previewUser, index) in previewUsers"
+					:key="previewUser.isUser ? previewUser.uid : `${previewUser.fullName}-${index}`"
+					class="contactsmenu__trigger-avatars__avatar"
+					:style="{ zIndex: previewUsers.length - index }"
+					:user="previewUser.isUser ? previewUser.uid : undefined"
+					:is-no-user="!previewUser.isUser"
+					:display-name="previewUser.fullName"
+					:size="32"
+					disable-menu
+					disable-tooltip
+					hide-status />
+			</span>
+			<NcIconSvgWrapper
+				v-else
+				class="contactsmenu__trigger-icon"
+				:path="mdiContacts" />
 		</template>
 		<div class="contactsmenu__menu">
 			<div class="contactsmenu__menu__search-container">
@@ -242,12 +291,66 @@ const userTeams: ITeam[] = []
 
 <style lang="scss" scoped>
 .contactsmenu {
-	overflow-y: hidden;
+	margin-inline-end: calc(2 * var(--default-grid-baseline));
+
+	:deep(.header-menu__trigger) {
+		opacity: 1 !important;
+
+		.button-vue__icon:has(.contactsmenu__trigger-avatars) {
+			mask: none !important;
+		}
+	}
+
+	&--avatar-stack {
+		width: fit-content !important;
+		min-width: var(--header-height);
+		overflow: visible;
+		flex-shrink: 0;
+
+		:deep(.header-menu__trigger) {
+			width: fit-content !important;
+			min-width: var(--header-height);
+			max-width: none;
+			overflow: visible !important;
+			padding-inline: var(--default-grid-baseline);
+
+			.button-vue__wrapper {
+				width: auto;
+				justify-content: center;
+			}
+
+			.button-vue__icon {
+				width: auto !important;
+				min-width: 0;
+				max-width: none;
+				height: auto;
+				min-height: 0;
+				overflow: visible;
+			}
+		}
+	}
 
 	&__trigger-icon {
 		color: var(--color-background-plain-text) !important;
 	}
 
+	&__trigger-avatars {
+		display: flex;
+		align-items: center;
+		pointer-events: none;
+
+		&__avatar {
+			box-sizing: content-box;
+			flex-shrink: 0;
+			border: 2px solid var(--color-background-plain);
+			margin-inline-start: -12px;
+
+			&:first-child {
+				margin-inline-start: 0;
+			}
+		}
+	}
+
 	&__menu {
 		display: flex;
 		flex-direction: column;

dist/core-main.js

@@ -125,4 +125,78 @@ public function testFindOne404(): void {
 		$this->assertEquals([], $response->getData());
 		$this->assertEquals(404, $response->getStatus());
 	}
+
+	public function testPreviewAvatarsWithoutTeam(): void {
+		$user = $this->createMock(IUser::class);
+		$contacts = [
+			$this->createMock(IEntry::class),
+			$this->createMock(IEntry::class),
+			$this->createMock(IEntry::class),
+			$this->createMock(IEntry::class),
+		];
+
+		$this->userSession->expects($this->once())
+			->method('getUser')
+			->willReturn($user);
+		$this->contactsManager->expects($this->once())
+			->method('getEntries')
+			->with($user, '')
+			->willReturn([
+				'contacts' => $contacts,
+				'contactsAppEnabled' => true,
+			]);
+		$this->teamManager->expects($this->never())
+			->method('getMembersOfTeam');
+
+		$this->assertEquals([
+			$contacts[0],
+			$contacts[1],
+			$contacts[2],
+		], $this->controller->previewAvatars());
+	}
+
+	public function testPreviewAvatarsWithTeam(): void {
+		$user = $this->createMock(IUser::class);
+		$user->method('getUID')->willReturn('current-user');
+
+		$alice = $this->createMock(IEntry::class);
+		$alice->method('getProperty')->with('UID')->willReturn('alice');
+		$external = $this->createMock(IEntry::class);
+		$external->method('getProperty')->with('UID')->willReturn('contact-1');
+		$bob = $this->createMock(IEntry::class);
+		$bob->method('getProperty')->with('UID')->willReturn('bob');
+		$carol = $this->createMock(IEntry::class);
+		$carol->method('getProperty')->with('UID')->willReturn('carol');
+
+		$this->userSession->expects($this->once())
+			->method('getUser')
+			->willReturn($user);
+		$this->contactsManager->expects($this->once())
+			->method('getEntries')
+			->with($user, '')
+			->willReturn([
+				'contacts' => [$alice, $external, $bob, $carol],
+				'contactsAppEnabled' => true,
+			]);
+		$this->teamManager->expects($this->once())
+			->method('getMembersOfTeam')
+			->with('team-id', 'current-user')
+			->willReturn([
+				'alice' => 'Alice',
+				'bob' => 'Bob',
+				'carol' => 'Carol',
+			]);
+
+		$this->assertEquals([$alice, $bob, $carol], $this->controller->previewAvatars('team-id'));
+	}
+
+	public function testPreviewAvatarsWithoutUser(): void {
+		$this->userSession->expects($this->once())
+			->method('getUser')
+			->willReturn(null);
+		$this->contactsManager->expects($this->never())
+			->method('getEntries');
+
+		$this->assertEquals([], $this->controller->previewAvatars());
+	}
 }