From 3a8c436839d34bc99656166ad14a68538b4cb06a Mon Sep 17 00:00:00 2001
From: Dwindi Ramadhana <dwindown@DwMacMiniM4.local>
Date: Sun, 4 Jan 2026 12:30:46 +0700
Subject: [PATCH] fix: Sidebar positioning - remove inset-y-0 conflict

- Fixed sidebar to not use inset-y-0 (was overriding top offset)
- Mobile: fixed positioning with sidebarTopClass
- Desktop: lg:sticky for proper sticky behavior
---
 admin-spa/src/routes/Help/index.tsx | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/admin-spa/src/routes/Help/index.tsx b/admin-spa/src/routes/Help/index.tsx
index a4ef859..c59726b 100644
--- a/admin-spa/src/routes/Help/index.tsx
+++ b/admin-spa/src/routes/Help/index.tsx
@@ -96,14 +96,16 @@ export default function Help() {
                 {sidebarOpen ? <X className="w-5 h-5" /> : <Menu className="w-5 h-5" />}
             </Button>
 
-            {/* Sidebar - sticky, positioned below header */}
+            {/* Sidebar - sticky on desktop, fixed on mobile */}
             <aside
                 className={cn(
-                    "w-72 border-r bg-muted/30 flex-shrink-0",
-                    "fixed lg:sticky inset-y-0 left-0 z-40",
+                    "w-72 border-r bg-muted/30 flex-shrink-0 overflow-y-auto",
+                    // Mobile: fixed position covering from below header
+                    "fixed left-0 z-40",
                     sidebarTopClass,
                     sidebarHeightClass,
-                    "lg:block overflow-y-auto",
+                    // Desktop: sticky instead of fixed
+                    "lg:sticky",
                     sidebarOpen ? "block" : "hidden lg:block"
                 )}
             >