From d412e760ab19261503a2deaf2089a396e2c19b26 Mon Sep 17 00:00:00 2001
From: Justin Edmund <justin@jedmund.com>
Date: Sun, 30 Nov 2025 23:16:04 -0800
Subject: [PATCH] nav: show register/login when logged out, move
 guides/collection to menu

---
 src/lib/components/Navigation.svelte | 45 +++++++++++++++++++---------
 1 file changed, 31 insertions(+), 14 deletions(-)

diff --git a/src/lib/components/Navigation.svelte b/src/lib/components/Navigation.svelte
index 477eab67..6cb2b8cb 100644
--- a/src/lib/components/Navigation.svelte
+++ b/src/lib/components/Navigation.svelte
@@ -44,6 +44,7 @@
 	const collectionHref = $derived(localizeHref('/collection'))
 	const meHref = $derived(localizeHref('/me'))
 	const loginHref = $derived(localizeHref('/auth/login'))
+	const registerHref = $derived(localizeHref('/auth/register'))
 	const settingsHref = $derived(localizeHref('/settings'))
 	const databaseHref = $derived(localizeHref('/database'))
 	const newTeamHref = $derived(localizeHref('/teams/new'))
@@ -151,15 +152,15 @@
 			<li>
 				<a href={galleryHref} class:selected={isNavSelected(galleryHref)}>{m.nav_gallery()}</a>
 			</li>
-			<li><a href={guidesHref} class:selected={isNavSelected(guidesHref)}>Guides</a></li>
-			<li>
-				<a href={collectionHref} class:selected={isNavSelected(collectionHref)}
-					>{m.nav_collection()}</a
-				>
-			</li>
-
-			<li>
-				{#if isAuth}
+			{#if isAuth}
+				<!-- Authenticated: show Guides and Collection in nav -->
+				<li><a href={guidesHref} class:selected={isNavSelected(guidesHref)}>Guides</a></li>
+				<li>
+					<a href={collectionHref} class:selected={isNavSelected(collectionHref)}
+						>{m.nav_collection()}</a
+					>
+				</li>
+				<li>
 					<a
 						href={meHref}
 						class:selected={isProfileSelected}
@@ -178,12 +179,18 @@
 						{/if}
 						<span>{username}</span>
 					</a>
-				{:else}
-					<a href={loginHref} class:selected={isNavSelected(loginHref)} aria-label="Login"
-						>{m.nav_login()}</a
+				</li>
+			{:else}
+				<!-- Not authenticated: show Register and Log in -->
+				<li>
+					<a href={registerHref} class:selected={isNavSelected(registerHref)}
+						>{m.nav_register()}</a
 					>
-				{/if}
-			</li>
+				</li>
+				<li>
+					<a href={loginHref} class:selected={isNavSelected(loginHref)}>{m.nav_login()}</a>
+				</li>
+			{/if}
 
 			<li>
 				<DropdownMenu.Root>
@@ -193,6 +200,16 @@
 
 					<DropdownMenu.Portal>
 						<DropdownMenu.Content class="dropdown-content" sideOffset={5}>
+							{#if !isAuth}
+								<!-- Not authenticated: show Guides and Collection in dropdown -->
+								<DropdownItem>
+									<a href={guidesHref}>Guides</a>
+								</DropdownItem>
+								<DropdownItem>
+									<a href={collectionHref}>{m.nav_collection()}</a>
+								</DropdownItem>
+								<DropdownMenu.Separator class="dropdown-separator" />
+							{/if}
 							<DropdownItem>
 								<button onclick={() => (settingsModalOpen = true)}>
 									{m.nav_settings()}