From b43e7fd3a7f91d5d7efd4a2f65155ac1c8ec2d73 Mon Sep 17 00:00:00 2001
From: Mike Cao <moocao@gmail.com>
Date: Thu, 29 Jan 2026 00:37:29 -0800
Subject: [PATCH] Hide sidebar collapse button on mobile

onItemClick is only passed on mobile, so use it to detect mobile context

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 src/app/share/[slug]/[[...path]]/ShareNav.tsx | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/app/share/[slug]/[[...path]]/ShareNav.tsx b/src/app/share/[slug]/[[...path]]/ShareNav.tsx
index f4eade5e..c5989f28 100644
--- a/src/app/share/[slug]/[[...path]]/ShareNav.tsx
+++ b/src/app/share/[slug]/[[...path]]/ShareNav.tsx
@@ -162,11 +162,13 @@ export function ShareNav({
             </Row>
           </a>
         )}
-        <Button variant="quiet" onPress={() => onCollapse?.(!collapsed)}>
-          <Icon color="muted">
-            <PanelLeft />
-          </Icon>
-        </Button>
+        {!onItemClick && (
+          <Button variant="quiet" onPress={() => onCollapse?.(!collapsed)}>
+            <Icon color="muted">
+              <PanelLeft />
+            </Icon>
+          </Button>
+        )}
       </Row>
       {!collapsed && (
         <Column flexGrow={1} overflowY="auto">