From 11a072e43c176c48beef5838931c29140a67aa62 Mon Sep 17 00:00:00 2001
From: Liviu Burcusel <x.liviu@gmail.com>
Date: Tue, 20 Jan 2026 12:31:36 +0100
Subject: [PATCH 1/2] Sidebar is closed when links are clicked on mobile

---
 app/layouts/default/Sidebar.vue       | 13 +++++--
 app/layouts/default/SidebarFooter.vue | 15 ++++----
 package-lock.json                     |  4 +--
 package.json                          |  2 +-
 tests/layouts/default/Sidebar.test.ts | 51 +++++++++++++++++++++++++++
 5 files changed, 71 insertions(+), 14 deletions(-)

diff --git a/app/layouts/default/Sidebar.vue b/app/layouts/default/Sidebar.vue
index eed0477..637c0be 100644
--- a/app/layouts/default/Sidebar.vue
+++ b/app/layouts/default/Sidebar.vue
@@ -1,5 +1,6 @@
 <script setup lang="ts">
 import { computed } from "vue";
+import { useSidebar } from "~/components/ui/sidebar";
 import { useRuntimeConfig } from "#app";
 import { BookOpen, BookOpenText, ChevronRight, HandCoins, Settings2, SquareTerminal } from "lucide-vue-next";
 
@@ -99,6 +100,14 @@ const props = withDefaults(defineProps<SidebarLayoutProps>(), {
 const navMain = computed(() => props.navItems || data.navMain);
 
 const config = useRuntimeConfig();
+
+const { isMobile, setOpenMobile } = useSidebar();
+
+const closeSidebarOnMobile = () => {
+  if (isMobile.value) {
+    setOpenMobile(false);
+  }
+};
 </script>
 
 <template>
@@ -107,7 +116,7 @@ const config = useRuntimeConfig();
       <SidebarMenu>
         <SidebarMenuItem>
           <SidebarMenuButton size="lg" as-child>
-            <NuxtLink to="/">
+            <NuxtLink to="/" @click="closeSidebarOnMobile">
               <div
                 class="flex aspect-square size-8 items-center justify-center rounded-lg bg-sidebar-primary text-sidebar-primary-foreground"
               >
@@ -147,7 +156,7 @@ const config = useRuntimeConfig();
                 <SidebarMenuSub>
                   <SidebarMenuSubItem v-for="subItem in item.items" :key="subItem.title">
                     <SidebarMenuSubButton as-child>
-                      <NuxtLink :to="subItem.url">
+                      <NuxtLink :to="subItem.url" @click="closeSidebarOnMobile">
                         <span>{{ subItem.title }}</span>
                       </NuxtLink>
                     </SidebarMenuSubButton>
diff --git a/app/layouts/default/SidebarFooter.vue b/app/layouts/default/SidebarFooter.vue
index 3fcd70e..88e59ee 100644
--- a/app/layouts/default/SidebarFooter.vue
+++ b/app/layouts/default/SidebarFooter.vue
@@ -16,7 +16,7 @@ import { BadgeCheck, Bell, ChevronsUpDown, CreditCard, LogIn, LogOut } from "luc
 
 import type { User } from "better-auth";
 
-const { isMobile } = useSidebar();
+const { isMobile, setOpenMobile } = useSidebar();
 
 const props = defineProps<{ user?: User | null | undefined }>();
 
@@ -28,12 +28,9 @@ const userInititials = computed(() => {
     .join("");
 });
 
-const handleLogout = () => {
-  navigateTo("/auth/logout");
-};
-
-const handleLogin = () => {
-  navigateTo("/auth/login");
+const handleAuthNavigation = (action: string) => {
+  setOpenMobile(false);
+  navigateTo(`/auth/${action}`);
 };
 </script>
 
@@ -93,7 +90,7 @@ const handleLogin = () => {
                 </DropdownMenuItem>
               </DropdownMenuGroup>
               <DropdownMenuSeparator />
-              <DropdownMenuItem @click="handleLogout">
+              <DropdownMenuItem @click="handleAuthNavigation('logout')">
                 <LogOut />
                 Log out
               </DropdownMenuItem>
@@ -136,7 +133,7 @@ const handleLogin = () => {
                 </div>
               </DropdownMenuLabel>
               <DropdownMenuSeparator />
-              <DropdownMenuItem @click="handleLogin">
+              <DropdownMenuItem @click="handleAuthNavigation('login')">
                 <LogIn />
                 Log in
               </DropdownMenuItem>
diff --git a/package-lock.json b/package-lock.json
index 6f402d4..c53986c 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "glowing-fiesta",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "glowing-fiesta",
-      "version": "0.0.5",
+      "version": "0.0.6",
       "hasInstallScript": true,
       "dependencies": {
         "@pinia/nuxt": "^0.11.3",
diff --git a/package.json b/package.json
index 02eca3e..db10a0b 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "glowing-fiesta",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "type": "module",
   "private": true,
   "scripts": {
diff --git a/tests/layouts/default/Sidebar.test.ts b/tests/layouts/default/Sidebar.test.ts
index 71ced88..c726ea3 100644
--- a/tests/layouts/default/Sidebar.test.ts
+++ b/tests/layouts/default/Sidebar.test.ts
@@ -267,4 +267,55 @@ describe("SidebarLayout", () => {
     // and renders the menu items.
     expect(text).toContain("Playground");
   });
+
+  it("calls setOpenMobile(false) when clicking a sub-menu item on mobile", async () => {
+    const setOpenMobile = vi.fn();
+    useSidebarMock.mockReturnValue({
+      isMobile: ref(true),
+      state: ref("expanded"),
+      openMobile: ref(true),
+      setOpenMobile,
+    });
+
+    const wrapper = mount({
+      components: { SidebarLayout },
+      template: "<Suspense><SidebarLayout /></Suspense>",
+    });
+
+    await flushPromises();
+
+    // Find the link containing 'History' (from default data)
+    const link = wrapper.findAll("a").find((el) => el.text().includes("History"));
+
+    expect(link).toBeDefined();
+    await link!.trigger("click");
+
+    expect(setOpenMobile).toHaveBeenCalledTimes(1);
+    expect(setOpenMobile).toHaveBeenCalledWith(false);
+  });
+
+  it("does not call setOpenMobile(false) when clicking a sub-menu item on desktop", async () => {
+    const setOpenMobile = vi.fn();
+    useSidebarMock.mockReturnValue({
+      isMobile: ref(false),
+      state: ref("expanded"),
+      openMobile: ref(true),
+      setOpenMobile,
+    });
+
+    const wrapper = mount({
+      components: { SidebarLayout },
+      template: "<Suspense><SidebarLayout /></Suspense>",
+    });
+
+    await flushPromises();
+
+    // Find the link containing 'History' (from default data)
+    const link = wrapper.findAll("a").find((el) => el.text().includes("History"));
+
+    expect(link).toBeDefined();
+    await link!.trigger("click");
+
+    expect(setOpenMobile).not.toHaveBeenCalled();
+  });
 });
-- 
2.49.1


From 0560ff762112d0d9d57a731a6efa81720a42af77 Mon Sep 17 00:00:00 2001
From: Liviu Burcusel <x.liviu@gmail.com>
Date: Tue, 20 Jan 2026 12:38:02 +0100
Subject: [PATCH 2/2] Code smells fixes

---
 tests/layouts/default/Sidebar.test.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/layouts/default/Sidebar.test.ts b/tests/layouts/default/Sidebar.test.ts
index c726ea3..c902f22 100644
--- a/tests/layouts/default/Sidebar.test.ts
+++ b/tests/layouts/default/Sidebar.test.ts
@@ -288,7 +288,7 @@ describe("SidebarLayout", () => {
     const link = wrapper.findAll("a").find((el) => el.text().includes("History"));
 
     expect(link).toBeDefined();
-    await link!.trigger("click");
+    await link.trigger("click");
 
     expect(setOpenMobile).toHaveBeenCalledTimes(1);
     expect(setOpenMobile).toHaveBeenCalledWith(false);
@@ -314,7 +314,7 @@ describe("SidebarLayout", () => {
     const link = wrapper.findAll("a").find((el) => el.text().includes("History"));
 
     expect(link).toBeDefined();
-    await link!.trigger("click");
+    await link.trigger("click");
 
     expect(setOpenMobile).not.toHaveBeenCalled();
   });
-- 
2.49.1