From 835a1c2fcd7eaef3c1abc53bc6b34612b24ea6b2 Mon Sep 17 00:00:00 2001
From: Pierre Berger <PierreBerger@users.noreply.github.com>
Date: Thu, 23 Oct 2025 23:16:30 +0200
Subject: [PATCH] ui: add missing close handler to cancel button on
 PauseBlockingDialog

---
 client/src/components/nav-actions.tsx | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/client/src/components/nav-actions.tsx b/client/src/components/nav-actions.tsx
index e3a2e8a..ea32d38 100644
--- a/client/src/components/nav-actions.tsx
+++ b/client/src/components/nav-actions.tsx
@@ -143,7 +143,11 @@ function CheckForUpdate() {
   });
 }
 
-export default function PauseBlockingDialog() {
+export default function PauseBlockingDialog({
+  onClose
+}: {
+  onClose: () => void;
+}) {
   type PausedResponse = {
     paused: boolean;
     timeLeft: number;
@@ -268,7 +272,11 @@ export default function PauseBlockingDialog() {
           </div>
 
           <DialogFooter className="flex justify-end gap-2">
-            <Button variant="outline" className="border-gray-300">
+            <Button
+              variant="outline"
+              className="border-gray-300"
+              onClick={onClose}
+            >
               Cancel
             </Button>
             <Button
@@ -299,9 +307,13 @@ export default function PauseBlockingDialog() {
 export function NavActions() {
   const [isOpen, setIsOpen] = useState(false);
   const [DialogComponent, setDialogComponent] = useState<
-    null | (() => JSX.Element)
+    null | ((props: { onClose: () => void }) => JSX.Element)
   >(null);
 
+  const closeDialog = () => {
+    setDialogComponent(null);
+  };
+
   return (
     <div>
       <Popover open={isOpen} onOpenChange={setIsOpen}>
@@ -354,7 +366,7 @@ export function NavActions() {
             if (!open) setDialogComponent(null);
           }}
         >
-          <DialogComponent />
+          <DialogComponent onClose={closeDialog} />
         </Dialog>
       )}
     </div>