From dda92f566443d6dab357f2b9080b9bd26fb69baa Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Sun, 23 Nov 2025 07:28:52 +0000
Subject: [PATCH] fix: ensure eventid is removed from URL when drawer closes

- Replace onCloseEventDetail with handleDrawerOpenChange
- Handle all drawer close scenarios: button click, outside click, ESC key
- Remove eventid from URL whenever drawer is closed via any method
- Fixes issue where eventid remained in URL after closing drawer
---
 src/pages/repair/RepairAdmin.tsx | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/pages/repair/RepairAdmin.tsx b/src/pages/repair/RepairAdmin.tsx
index 9044c08..0054e79 100644
--- a/src/pages/repair/RepairAdmin.tsx
+++ b/src/pages/repair/RepairAdmin.tsx
@@ -411,14 +411,16 @@ export default function App() {
     window.history.replaceState({}, '', newUrl)
   }
 
-  const onCloseEventDetail = () => {
+  const handleDrawerOpenChange = (isOpen: boolean) => {
     onOpenChange()
 
-    // Remove eventid from URL
-    const params = new URLSearchParams(window.location.search)
-    params.delete('eventid')
-    const newUrl = `${window.location.pathname}?${params.toString()}`
-    window.history.replaceState({}, '', newUrl)
+    // Remove eventid from URL when drawer is closed
+    if (!isOpen) {
+      const params = new URLSearchParams(window.location.search)
+      params.delete('eventid')
+      const newUrl = `${window.location.pathname}?${params.toString()}`
+      window.history.replaceState({}, '', newUrl)
+    }
   }
 
   const MobileEventCard = ({ event }: { event: PublicEvent }) => (
@@ -617,8 +619,8 @@ export default function App() {
           }
         }
         isOpen={isOpen}
-        onOpenChange={onOpenChange}
-        onClose={onCloseEventDetail}
+        onOpenChange={handleDrawerOpenChange}
+        onClose={() => handleDrawerOpenChange(false)}
         onDelete={() => {}}
         onEdit={() => {}}
       >