Fix mobile nav menu context scoping issue

edgedb · Sep 10, 2024 · 3e139df · 3e139df
1 parent e46ce03
commit 3e139df
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/shared/common/ui/navtabs/mobile.tsx b/shared/common/ui/navtabs/mobile.tsx
@@ -22,7 +22,14 @@ export function MobileNavTabs({
   Link,
   extraMenu,
 }: MobileNavTabsProps) {
-  const {openModal} = useModal();
+  const {modal, openModal} = useModal(
+    <MobileMenu
+      tabs={tabs}
+      currentTabId={currentTabId}
+      Link={Link}
+      extraMenu={extraMenu}
+    />
+  );
   const modalDisposer = useRef<null | (() => void)>(null);
   const windowWidth = useWindowWidth();
 
@@ -54,19 +61,11 @@ export function MobileNavTabs({
       </div>
       <div
         className={styles.menuButton}
-        onClick={() =>
-          (modalDisposer.current = openModal(
-            <MobileMenu
-              tabs={tabs}
-              currentTabId={currentTabId}
-              Link={Link}
-              extraMenu={extraMenu}
-            />
-          ))
-        }
+        onClick={() => (modalDisposer.current = openModal())}
       >
         <EllipsisIcon />
       </div>
+      {modal}
     </div>
   );
 }