From 27163e3d72c95b685fd8baaee9ad1845abb392ed Mon Sep 17 00:00:00 2001 From: Marcel Raddatz Date: Fri, 10 Apr 2026 17:32:37 +0200 Subject: [PATCH] feat(nav): remove Mitglieder link from desktop sidebar Co-Authored-By: Claude Sonnet 4.6 --- frontend/src/lib/nav/AppShell.test.ts | 4 ++-- frontend/src/lib/nav/DesktopSidebar.test.ts | 8 ++++---- frontend/src/lib/nav/nav.test.ts | 4 ++-- frontend/src/lib/nav/nav.ts | 1 - 4 files changed, 8 insertions(+), 9 deletions(-) diff --git a/frontend/src/lib/nav/AppShell.test.ts b/frontend/src/lib/nav/AppShell.test.ts index 24e09b4..bbfa139 100644 --- a/frontend/src/lib/nav/AppShell.test.ts +++ b/frontend/src/lib/nav/AppShell.test.ts @@ -31,7 +31,7 @@ describe('AppShell', () => { it('renders all navigation links from all nav variants', () => { render(AppShell, { props: defaultProps }); const links = screen.getAllByRole('link'); - // Mobile: 4, Tablet: 4, Desktop: 5 = 13 total - expect(links).toHaveLength(13); + // Mobile: 4, Tablet: 4, Desktop: 4 = 12 total + expect(links).toHaveLength(12); }); }); diff --git a/frontend/src/lib/nav/DesktopSidebar.test.ts b/frontend/src/lib/nav/DesktopSidebar.test.ts index b784791..b5bf988 100644 --- a/frontend/src/lib/nav/DesktopSidebar.test.ts +++ b/frontend/src/lib/nav/DesktopSidebar.test.ts @@ -28,17 +28,17 @@ describe('DesktopSidebar', () => { expect(screen.getByText('Einkauf')).toBeInTheDocument(); }); - it('renders Household section with 2 items', () => { + it('renders Household section with Einstellungen', () => { render(DesktopSidebar, { props: { appName: 'Mealprep', householdName: 'Familie Müller' } }); expect(screen.getByText('Haushalt')).toBeInTheDocument(); - expect(screen.getByText('Mitglieder')).toBeInTheDocument(); expect(screen.getByText('Einstellungen')).toBeInTheDocument(); + expect(screen.queryByText('Mitglieder')).not.toBeInTheDocument(); }); - it('has 5 navigation links total', () => { + it('has 4 navigation links total', () => { render(DesktopSidebar, { props: { appName: 'Mealprep', householdName: 'Familie Müller' } }); const links = screen.getAllByRole('link'); - expect(links).toHaveLength(5); + expect(links).toHaveLength(4); }); it('marks active item with aria-current="page"', () => { diff --git a/frontend/src/lib/nav/nav.test.ts b/frontend/src/lib/nav/nav.test.ts index eff19e5..62b579e 100644 --- a/frontend/src/lib/nav/nav.test.ts +++ b/frontend/src/lib/nav/nav.test.ts @@ -34,9 +34,9 @@ describe('nav config', () => { expect(labels).toEqual(['Planer', 'Rezepte', 'Einkauf']); }); - it('Household section has Members, Settings', () => { + it('Household section has Settings', () => { const labels = desktopNavSections[1].items.map((item) => item.label); - expect(labels).toEqual(['Mitglieder', 'Einstellungen']); + expect(labels).toEqual(['Einstellungen']); }); }); diff --git a/frontend/src/lib/nav/nav.ts b/frontend/src/lib/nav/nav.ts index 76657e6..16fc317 100644 --- a/frontend/src/lib/nav/nav.ts +++ b/frontend/src/lib/nav/nav.ts @@ -37,7 +37,6 @@ export const desktopNavSections: NavSection[] = [ { title: 'Haushalt', items: [ - { href: '/members', label: 'Mitglieder', icon: '👥' }, { href: '/settings', label: 'Einstellungen', icon: '⚙️', extraPaths: ['/household/staples'] } ] }