@getmicdrop/svelte-components 5.17.1 → 5.17.3

package/dist/patterns/navigation/BottomNav.stories.svelte

@@ -1,117 +1,117 @@
-<script module>
-  import { defineMeta } from "@storybook/addon-svelte-csf";
-
-  import BottomNav from './BottomNav.svelte';
-
-  const { Story } = defineMeta({
-    title: 'Components/Layout/BottomNav',
-    component: BottomNav,
-    tags: ['autodocs'],
-  });
-
-</script>
-
-<script>
-  
-  const defaultLinks = [
-    { label: 'Home', href: '/', icon: 'Home' },
-    { label: 'Shows', href: '/shows', icon: 'Shows' },
-    { label: 'Availability', href: '/availability', icon: 'Availability' },
-    { label: 'Profile', href: '/profile', icon: 'Profile' },
-  ];
-
-  const extendedLinks = [
-    { label: 'Home', href: '/', icon: 'Home' },
-    { label: 'Shows', href: '/shows', icon: 'Shows' },
-    { label: 'Availability', href: '/availability', icon: 'Availability' },
-    { label: 'Messages', href: '/messages', icon: 'Message' },
-    { label: 'Profile', href: '/profile', icon: 'Profile' },
-  ];
-</script>
-
-<Story name="Default (4 Tabs)">
-  <div class="bg-gray-100 dark:bg-gray-700 p-4">
-    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Mobile Bottom Navigation (4 tabs)</p>
-    <div class="max-w-md mx-auto bg-white dark:bg-gray-900 rounded-lg shadow-lg overflow-hidden">
-      <div class="h-64 bg-gradient-to-br from-blue-50 to-blue-100 flex items-center justify-center">
-        <p class="text-gray-500 dark:text-gray-400">Page Content Area</p>
-      </div>
-      <div class="flex border-t bg-white dark:bg-gray-900">
-        <BottomNav links={defaultLinks} />
-      </div>
-    </div>
-  </div>
-</Story>
-
-<Story name="With 5 Tabs">
-  <div class="bg-gray-100 dark:bg-gray-700 p-4">
-    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Extended Navigation (5 tabs)</p>
-    <div class="max-w-md mx-auto bg-white dark:bg-gray-900 rounded-lg shadow-lg overflow-hidden">
-      <div class="h-64 bg-gradient-to-br from-purple-50 to-purple-100 flex items-center justify-center">
-        <p class="text-gray-500 dark:text-gray-400">Page Content Area</p>
-      </div>
-      <div class="flex border-t bg-white dark:bg-gray-900">
-        <BottomNav links={extendedLinks} />
-      </div>
-    </div>
-  </div>
-</Story>
-
-<Story name="Different Icon Set">
-  <div class="bg-gray-100 dark:bg-gray-700 p-4">
-    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Custom Icons</p>
-    <div class="max-w-md mx-auto bg-white dark:bg-gray-900 rounded-lg shadow-lg overflow-hidden">
-      <div class="h-64 bg-gradient-to-br from-green-50 to-green-100 flex items-center justify-center">
-        <p class="text-gray-500 dark:text-gray-400">Page Content Area</p>
-      </div>
-      <div class="flex border-t bg-white dark:bg-gray-900">
-        <BottomNav
-          links={[
-            { label: 'Dashboard', href: '/', icon: 'Home' },
-            { label: 'Events', href: '/events', icon: 'Shows' },
-            { label: 'Notifications', href: '/notifications', icon: 'Notification' },
-            { label: 'Settings', href: '/settings', icon: 'Profile' },
-          ]}
-        />
-      </div>
-    </div>
-  </div>
-</Story>
-
-<Story name="Mobile Viewport">
-  <div class="max-w-[375px] mx-auto bg-white dark:bg-gray-900 shadow-2xl rounded-3xl overflow-hidden">
-    <!-- Mock iPhone screen -->
-    <div class="h-[667px] flex flex-col">
-      <!-- Status bar -->
-      <div class="h-6 bg-black flex items-center justify-between px-4">
-        <span class="text-white text-xs">9:41</span>
-        <span class="text-white text-xs">100%</span>
-      </div>
-
-      <!-- Content area -->
-      <div class="flex-1 bg-gradient-to-br from-blue-50 to-purple-50 p-4">
-        <h1 class="text-2xl font-bold mb-2">Home</h1>
-        <p class="text-gray-500 dark:text-gray-400">This is a mobile viewport simulation</p>
-      </div>
-
-      <!-- Bottom navigation -->
-      <div class="flex border-t bg-white dark:bg-gray-900 safe-bottom">
-        <BottomNav links={defaultLinks} />
-      </div>
-    </div>
-  </div>
-</Story>
-
-<Story name="Dark Mode">
-  <div class="dark bg-gray-900 p-4 min-h-screen">
-    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Dark Mode Bottom Navigation</p>
-    <div class="max-w-md mx-auto bg-gray-800 rounded-lg shadow-2xl overflow-hidden">
-      <div class="h-64 bg-gradient-to-br from-gray-900 to-gray-800 flex items-center justify-center">
-        <p class="text-gray-500 dark:text-gray-400">Dark Mode Content</p>
-      </div>
-      <div class="flex border-t border-gray-700 bg-gray-800">
-        <BottomNav links={defaultLinks} />
-      </div>
-    </div>
-  </div>
-</Story>
+<script module>
+  import { defineMeta } from "@storybook/addon-svelte-csf";
+
+  import BottomNav from './BottomNav.svelte';
+
+  const { Story } = defineMeta({
+    title: 'Components/Layout/BottomNav',
+    component: BottomNav,
+    tags: ['autodocs'],
+  });
+
+</script>
+
+<script>
+  
+  const defaultLinks = [
+    { label: 'Home', href: '/', icon: 'Home' },
+    { label: 'Shows', href: '/shows', icon: 'Shows' },
+    { label: 'Availability', href: '/availability', icon: 'Availability' },
+    { label: 'Profile', href: '/profile', icon: 'Profile' },
+  ];
+
+  const extendedLinks = [
+    { label: 'Home', href: '/', icon: 'Home' },
+    { label: 'Shows', href: '/shows', icon: 'Shows' },
+    { label: 'Availability', href: '/availability', icon: 'Availability' },
+    { label: 'Messages', href: '/messages', icon: 'Message' },
+    { label: 'Profile', href: '/profile', icon: 'Profile' },
+  ];
+</script>
+
+<Story name="Default (4 Tabs)">
+  <div class="bg-gray-100 dark:bg-gray-700 p-4">
+    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Mobile Bottom Navigation (4 tabs)</p>
+    <div class="max-w-md mx-auto bg-white dark:bg-gray-900 rounded-lg shadow-lg overflow-hidden">
+      <div class="h-64 bg-gradient-to-br from-blue-50 to-blue-100 flex items-center justify-center">
+        <p class="text-gray-500 dark:text-gray-400">Page Content Area</p>
+      </div>
+      <div class="flex border-t bg-white dark:bg-gray-900">
+        <BottomNav links={defaultLinks} />
+      </div>
+    </div>
+  </div>
+</Story>
+
+<Story name="With 5 Tabs">
+  <div class="bg-gray-100 dark:bg-gray-700 p-4">
+    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Extended Navigation (5 tabs)</p>
+    <div class="max-w-md mx-auto bg-white dark:bg-gray-900 rounded-lg shadow-lg overflow-hidden">
+      <div class="h-64 bg-gradient-to-br from-purple-50 to-purple-100 flex items-center justify-center">
+        <p class="text-gray-500 dark:text-gray-400">Page Content Area</p>
+      </div>
+      <div class="flex border-t bg-white dark:bg-gray-900">
+        <BottomNav links={extendedLinks} />
+      </div>
+    </div>
+  </div>
+</Story>
+
+<Story name="Different Icon Set">
+  <div class="bg-gray-100 dark:bg-gray-700 p-4">
+    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Custom Icons</p>
+    <div class="max-w-md mx-auto bg-white dark:bg-gray-900 rounded-lg shadow-lg overflow-hidden">
+      <div class="h-64 bg-gradient-to-br from-green-50 to-green-100 flex items-center justify-center">
+        <p class="text-gray-500 dark:text-gray-400">Page Content Area</p>
+      </div>
+      <div class="flex border-t bg-white dark:bg-gray-900">
+        <BottomNav
+          links={[
+            { label: 'Dashboard', href: '/', icon: 'Home' },
+            { label: 'Events', href: '/events', icon: 'Shows' },
+            { label: 'Notifications', href: '/notifications', icon: 'Notification' },
+            { label: 'Settings', href: '/settings', icon: 'Profile' },
+          ]}
+        />
+      </div>
+    </div>
+  </div>
+</Story>
+
+<Story name="Mobile Viewport">
+  <div class="max-w-[375px] mx-auto bg-white dark:bg-gray-900 shadow-2xl rounded-3xl overflow-hidden">
+    <!-- Mock iPhone screen -->
+    <div class="h-[667px] flex flex-col">
+      <!-- Status bar -->
+      <div class="h-6 bg-black flex items-center justify-between px-4">
+        <span class="text-white text-xs">9:41</span>
+        <span class="text-white text-xs">100%</span>
+      </div>
+
+      <!-- Content area -->
+      <div class="flex-1 bg-gradient-to-br from-blue-50 to-purple-50 p-4">
+        <h1 class="text-2xl font-bold mb-2">Home</h1>
+        <p class="text-gray-500 dark:text-gray-400">This is a mobile viewport simulation</p>
+      </div>
+
+      <!-- Bottom navigation -->
+      <div class="flex border-t bg-white dark:bg-gray-900 safe-bottom">
+        <BottomNav links={defaultLinks} />
+      </div>
+    </div>
+  </div>
+</Story>
+
+<Story name="Dark Mode">
+  <div class="dark bg-gray-900 p-4 min-h-screen">
+    <p class="text-sm text-gray-500 dark:text-gray-400 mb-4">Dark Mode Bottom Navigation</p>
+    <div class="max-w-md mx-auto bg-gray-800 rounded-lg shadow-2xl overflow-hidden">
+      <div class="h-64 bg-gradient-to-br from-gray-900 to-gray-800 flex items-center justify-center">
+        <p class="text-gray-500 dark:text-gray-400">Dark Mode Content</p>
+      </div>
+      <div class="flex border-t border-gray-700 bg-gray-800">
+        <BottomNav links={defaultLinks} />
+      </div>
+    </div>
+  </div>
+</Story>

package/dist/patterns/navigation/BottomNav.svelte

@@ -1,74 +1,74 @@
-<script lang="ts">
-  import { page } from '../../__LIB_STORES__.js';
-  import { base } from '../../__LIB_PATHS__.js';
-  import { browser } from '../../__LIB_ENVIRONMENT__.js';
-  import { onMount } from "svelte";
-  import Icon from "../../primitives/Icons/Icon.svelte";
-  import { typography } from '../../tokens/typography';
-
-  interface NavLink {
-    label: string;
-    href: string;
-    icon: string;
-  }
-
-  interface Props {
-    links?: NavLink[];
-  }
-
-  let {
-    links = [],
-  }: Props = $props();
-
-  let invitationCount = $state(0);
-
-  let currentPath = $derived($page.url.pathname);
-
-  function isActive(href: string, path: string): boolean {
-    if (href === `${base}/` || href === '/') {
-      return path === href || path === `${base}/`;
-    }
-    return path.startsWith(href);
-  }
-
-  onMount(() => {
-    async function fetchShows() {
-      try {
-        const res = await fetch(`/api/getEventsForVenue`);
-        if (!res.ok) {
-          // Silently ignore auth errors - user may not be logged in yet
-          if (res.status === 401) return;
-          return;
-        }
-        const events = await res.json();
-        invitationCount = events.filter((event: { acceptedState: number }) => event.acceptedState === 0).length;
-      } catch (_err) {
-        // Silently fail - this is non-critical UI enhancement
-      }
-    }
-
-    if (browser) {
-      fetchShows();
-    }
-  });
-</script>
-
-{#each links as { label, href, icon }}
-  <a
-    {href}
-    class={`bottom-nav-item flex-1 flex items-center justify-center py-3 px-2 no-underline bg-transparent border-none cursor-pointer min-w-0
-      ${typography.textMuted} transition-colors duration-150
-      hover:text-gray-900 hover:no-underline dark:hover:text-white
-      select-none touch-manipulation
-      ${isActive(href, currentPath) ? 'active text-blue-700 dark:text-blue-500' : ''}`}
-    aria-label={label}
-    style="-webkit-tap-highlight-color: transparent; -webkit-touch-callout: none;"
-  >
-    <span class="bottom-nav-icon relative flex items-center justify-center transition-transform duration-150 active:scale-[0.8]">
-      <Icon name={icon} size="24" />
-      {#if label === "Shows" && invitationCount > 0}
-        <span class="notification-dot absolute -top-1 -right-1 w-2 h-2 bg-red-500 rounded-full"></span>
-      {/if}
-    </span>
-  </a>
-{/each}
+<script lang="ts">
+  import { page } from '../../__LIB_STORES__.js';
+  import { base } from '../../__LIB_PATHS__.js';
+  import { browser } from '../../__LIB_ENVIRONMENT__.js';
+  import { onMount } from "svelte";
+  import Icon from "../../primitives/Icons/Icon.svelte";
+  import { typography } from '../../tokens/typography';
+
+  interface NavLink {
+    label: string;
+    href: string;
+    icon: string;
+  }
+
+  interface Props {
+    links?: NavLink[];
+  }
+
+  let {
+    links = [],
+  }: Props = $props();
+
+  let invitationCount = $state(0);
+
+  let currentPath = $derived($page.url.pathname);
+
+  function isActive(href: string, path: string): boolean {
+    if (href === `${base}/` || href === '/') {
+      return path === href || path === `${base}/`;
+    }
+    return path.startsWith(href);
+  }
+
+  onMount(() => {
+    async function fetchShows() {
+      try {
+        const res = await fetch(`/api/getEventsForVenue`);
+        if (!res.ok) {
+          // Silently ignore auth errors - user may not be logged in yet
+          if (res.status === 401) return;
+          return;
+        }
+        const events = await res.json();
+        invitationCount = events.filter((event: { acceptedState: number }) => event.acceptedState === 0).length;
+      } catch (_err) {
+        // Silently fail - this is non-critical UI enhancement
+      }
+    }
+
+    if (browser) {
+      fetchShows();
+    }
+  });
+</script>
+
+{#each links as { label, href, icon }}
+  <a
+    {href}
+    class={`bottom-nav-item flex-1 flex items-center justify-center py-3 px-2 no-underline bg-transparent border-none cursor-pointer min-w-0
+      ${typography.textMuted} transition-colors duration-150
+      hover:text-gray-900 hover:no-underline dark:hover:text-white
+      select-none touch-manipulation
+      ${isActive(href, currentPath) ? 'active text-blue-700 dark:text-blue-500' : ''}`}
+    aria-label={label}
+    style="-webkit-tap-highlight-color: transparent; -webkit-touch-callout: none;"
+  >
+    <span class="bottom-nav-icon relative flex items-center justify-center transition-transform duration-150 active:scale-[0.8]">
+      <Icon name={icon} size="24" />
+      {#if label === "Shows" && invitationCount > 0}
+        <span class="notification-dot absolute -top-1 -right-1 w-2 h-2 bg-red-500 rounded-full"></span>
+      {/if}
+    </span>
+  </a>
+{/each}

package/dist/patterns/navigation/Header.spec.js

@@ -136,6 +136,18 @@ describe('Header', () => {
             await fireEvent.click(desktopButton);
             expect(container.querySelector('div.absolute.top-\\[calc\\(100\\%\\+0\\.5rem\\)\\]')).toBeFalsy();
         });
+        it('navigates and closes dropdown when a dropdown link is clicked', async () => {
+            const { goto } = await import('$app/navigation');
+            const { container } = render(Header, defaultProps);
+            const desktopButton = container.querySelector('button.hidden.md\\:block');
+            await fireEvent.click(desktopButton);
+            // Click the "Account Settings" menu item in the dropdown
+            const accountSettings = screen.getByText('Account Settings');
+            await fireEvent.click(accountSettings);
+            expect(goto).toHaveBeenCalledWith('/account-settings');
+            // Dropdown should be closed after clicking
+            expect(container.querySelector('div.absolute.top-\\[calc\\(100\\%\\+0\\.5rem\\)\\]')).toBeFalsy();
+        });
     });
     describe('mobile sheet', () => {
         it('triggers mobile sheet state when mobile avatar is clicked', async () => {
@@ -150,6 +162,69 @@ describe('Header', () => {
             const mobileSheet = document.body.querySelector('div.fixed.inset-0.bg-black\\/50');
             expect(mobileSheet).toBeTruthy();
         });
+        it('navigates when a mobile sheet dropdown link is clicked', async () => {
+            const { goto } = await import('$app/navigation');
+            const { container } = render(Header, defaultProps);
+            const mobileButton = container.querySelector('button.block.md\\:hidden');
+            await fireEvent.click(mobileButton);
+            // Mobile sheet renders via portal in document.body
+            const mobileSheet = document.body.querySelector('div.fixed.inset-0.bg-black\\/50');
+            expect(mobileSheet).toBeTruthy();
+            // Click the first dropdown link in the mobile sheet
+            const menuItems = mobileSheet.querySelectorAll('button');
+            // Find the menu item with "Account Settings" text
+            const accountBtn = Array.from(menuItems).find(btn => btn.textContent?.includes('Account Settings'));
+            expect(accountBtn).toBeTruthy();
+            await fireEvent.click(accountBtn);
+            expect(goto).toHaveBeenCalledWith('/account-settings');
+        });
+        it('calls signoutHandler and closes sheet when mobile sign out is clicked', async () => {
+            const signoutHandler = vi.fn();
+            const { container } = render(Header, { ...defaultProps, signoutHandler });
+            const mobileButton = container.querySelector('button.block.md\\:hidden');
+            await fireEvent.click(mobileButton);
+            const mobileSheet = document.body.querySelector('div.fixed.inset-0.bg-black\\/50');
+            expect(mobileSheet).toBeTruthy();
+            // Find the Sign Out button in the mobile sheet
+            const menuItems = mobileSheet.querySelectorAll('button');
+            const signOutBtn = Array.from(menuItems).find(btn => btn.textContent?.includes('Sign Out'));
+            expect(signOutBtn).toBeTruthy();
+            await fireEvent.click(signOutBtn);
+            expect(signoutHandler).toHaveBeenCalled();
+        });
+    });
+    describe('back button mode desktop dropdown', () => {
+        it('shows dropdown with dropdown links and sign out when avatar clicked in back-button mode', async () => {
+            const { goto } = await import('$app/navigation');
+            const signoutHandler = vi.fn();
+            const { container } = render(Header, { ...defaultProps, showBackButton: true, signoutHandler });
+            // In back-button mode, the desktop section is within "hidden md:flex"
+            // The avatar button is the only AvatarButton in that section
+            const avatarButtons = container.querySelectorAll('button');
+            // Find the avatar button that opens the dropdown (not the Back button)
+            const desktopAvatarBtn = Array.from(avatarButtons).find(btn => !btn.textContent?.includes('Back') && btn.querySelector('img'));
+            expect(desktopAvatarBtn).toBeTruthy();
+            await fireEvent.click(desktopAvatarBtn);
+            // Dropdown should be visible
+            const dropdown = container.querySelector('div.absolute.top-\\[calc\\(100\\%\\+0\\.5rem\\)\\]');
+            expect(dropdown).toBeTruthy();
+            // Click a dropdown link
+            const accountSettings = screen.getByText('Account Settings');
+            await fireEvent.click(accountSettings);
+            expect(goto).toHaveBeenCalledWith('/account-settings');
+        });
+        it('calls signoutHandler when sign out clicked in back-button mode dropdown', async () => {
+            const signoutHandler = vi.fn();
+            const { container } = render(Header, { ...defaultProps, showBackButton: true, signoutHandler });
+            // Find the avatar button (not the Back button)
+            const avatarButtons = container.querySelectorAll('button');
+            const desktopAvatarBtn = Array.from(avatarButtons).find(btn => !btn.textContent?.includes('Back') && btn.querySelector('img'));
+            await fireEvent.click(desktopAvatarBtn);
+            // Click sign out
+            const signOutBtn = screen.getByText('Sign out');
+            await fireEvent.click(signOutBtn);
+            expect(signoutHandler).toHaveBeenCalled();
+        });
     });
     describe('navigation active state', () => {
         it('marks current path as active', () => {
@@ -166,5 +241,53 @@ describe('Header', () => {
             await fireEvent.click(desktopButton);
             expect(screen.getByText('Performer')).toBeTruthy();
         });
+        it('shows name in mobile sheet when provided', async () => {
+            const { container } = render(Header, { ...defaultProps, name: 'Jane Doe' });
+            const mobileButton = container.querySelector('button.block.md\\:hidden');
+            await fireEvent.click(mobileButton);
+            const mobileSheet = document.body.querySelector('div.fixed.inset-0.bg-black\\/50');
+            expect(mobileSheet).toBeTruthy();
+            expect(mobileSheet.textContent).toContain('Jane Doe');
+        });
+        it('shows "Performer" in mobile sheet when name is empty', async () => {
+            const { container } = render(Header, { ...defaultProps, name: '' });
+            const mobileButton = container.querySelector('button.block.md\\:hidden');
+            await fireEvent.click(mobileButton);
+            const mobileSheet = document.body.querySelector('div.fixed.inset-0.bg-black\\/50');
+            expect(mobileSheet).toBeTruthy();
+            expect(mobileSheet.textContent).toContain('Performer');
+        });
+        it('shows "Performer" in back-button mode dropdown when name is empty', async () => {
+            const { container } = render(Header, { ...defaultProps, name: '', showBackButton: true });
+            const avatarButtons = container.querySelectorAll('button');
+            const desktopAvatarBtn = Array.from(avatarButtons).find(btn => !btn.textContent?.includes('Back') && btn.querySelector('img'));
+            await fireEvent.click(desktopAvatarBtn);
+            const dropdown = container.querySelector('div.absolute.top-\\[calc\\(100\\%\\+0\\.5rem\\)\\]');
+            expect(dropdown).toBeTruthy();
+            expect(dropdown.textContent).toContain('Performer');
+        });
+    });
+    describe('isActive edge cases', () => {
+        it('marks root nav link as active when path matches base', () => {
+            // The mock sets page pathname to '/shows' and base to '/performers'
+            // Add a root link and verify it is NOT active (since path is /shows)
+            const propsWithRoot = {
+                ...defaultProps,
+                navLinks: [
+                    { label: 'Home', href: '/performers/' },
+                    ...defaultProps.navLinks
+                ]
+            };
+            const { container } = render(Header, propsWithRoot);
+            const homeLink = container.querySelector('a[href="/performers/"]');
+            // /shows does not match /performers/, so it should NOT be active
+            expect(homeLink.classList.contains('text-blue-700')).toBe(false);
+        });
+        it('marks link as active using startsWith for non-root paths', () => {
+            const { container } = render(Header, defaultProps);
+            // '/shows' starts with '/shows' so it should be active
+            const showsLink = container.querySelector('a[href="/shows"]');
+            expect(showsLink.classList.contains('text-blue-700')).toBe(true);
+        });
     });
 });