From 04514d21894d9025ac13808a86b906a9596aec0b Mon Sep 17 00:00:00 2001
From: msinha569 <msinha569@gmail.com>
Date: Thu, 30 Oct 2025 14:42:53 +0530
Subject: [PATCH 1/2] Fix: make entire sidebar button clickable

---
 .../components/sidebar/app-sidebar.tsx        | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/surfsense_web/components/sidebar/app-sidebar.tsx b/surfsense_web/components/sidebar/app-sidebar.tsx
index 116260032..a99e77c38 100644
--- a/surfsense_web/components/sidebar/app-sidebar.tsx
+++ b/surfsense_web/components/sidebar/app-sidebar.tsx
@@ -32,6 +32,7 @@ import {
 	SidebarMenuButton,
 	SidebarMenuItem,
 } from "@/components/ui/sidebar";
+import Link from "next/link";
 
 // Map of icon names to their components
 export const iconMap: Record<string, LucideIcon> = {
@@ -213,16 +214,16 @@ export const AppSidebar = memo(function AppSidebar({
 			<SidebarHeader>
 				<SidebarMenu>
 					<SidebarMenuItem>
-						<SidebarMenuButton size="lg" asChild aria-label="Go to home page">
-							<div>
-								<div className="bg-sidebar-primary text-sidebar-primary-foreground flex aspect-square size-8 items-center justify-center rounded-lg">
-									<Logo className="rounded-lg" />
-								</div>
-								<div className="grid flex-1 text-left text-sm leading-tight">
-									<span className="truncate font-medium">SurfSense</span>
-									<span className="truncate text-xs">beta v0.0.8</span>
-								</div>
+						<SidebarMenuButton asChild size="lg">
+						<Link href="/" className="flex items-center gap-2 w-full">
+							<div className="bg-sidebar-primary text-sidebar-primary-foreground flex aspect-square size-8 items-center justify-center rounded-lg">
+							<Logo className="pointer-events-none rounded-lg" />
 							</div>
+							<div className="grid flex-1 text-left text-sm leading-tight">
+							<span className="truncate font-medium">SurfSense</span>
+							<span className="truncate text-xs">beta v0.0.8</span>
+							</div>
+						</Link>
 						</SidebarMenuButton>
 					</SidebarMenuItem>
 				</SidebarMenu>

From 1cee0bc312b9632c95a6cd00f6bffbe721ae9489 Mon Sep 17 00:00:00 2001
From: msinha569 <msinha569@gmail.com>
Date: Thu, 30 Oct 2025 15:12:06 +0530
Subject: [PATCH 2/2] chore: fix import order as per Biome rules and removed
 nested links to avoid invalid HTML

---
 surfsense_web/components/sidebar/app-sidebar.tsx | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/surfsense_web/components/sidebar/app-sidebar.tsx b/surfsense_web/components/sidebar/app-sidebar.tsx
index a99e77c38..243f2c062 100644
--- a/surfsense_web/components/sidebar/app-sidebar.tsx
+++ b/surfsense_web/components/sidebar/app-sidebar.tsx
@@ -1,5 +1,8 @@
 "use client";
 
+import Link from "next/link";
+import Image from "next/image";
+
 import {
 	AlertCircle,
 	BookOpen,
@@ -32,7 +35,7 @@ import {
 	SidebarMenuButton,
 	SidebarMenuItem,
 } from "@/components/ui/sidebar";
-import Link from "next/link";
+
 
 // Map of icon names to their components
 export const iconMap: Record<string, LucideIcon> = {
@@ -217,7 +220,13 @@ export const AppSidebar = memo(function AppSidebar({
 						<SidebarMenuButton asChild size="lg">
 						<Link href="/" className="flex items-center gap-2 w-full">
 							<div className="bg-sidebar-primary text-sidebar-primary-foreground flex aspect-square size-8 items-center justify-center rounded-lg">
-							<Logo className="pointer-events-none rounded-lg" />
+							<Image
+								src="/icon-128.png"
+								alt="SurfSense logo"
+								width={32}
+								height={32}
+								className="rounded-lg"
+							/>
 							</div>
 							<div className="grid flex-1 text-left text-sm leading-tight">
 							<span className="truncate font-medium">SurfSense</span>