Skip to content

Commit cd90751

Browse files
committed
fix: X button not closing UserAllIPsModal
1 parent d2af441 commit cd90751

File tree

1 file changed

+3
-21
lines changed

1 file changed

+3
-21
lines changed

dashboard/src/components/dialogs/UserAllIPsModal.tsx

Lines changed: 3 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -310,31 +310,13 @@ export default function UserAllIPsModal({ isOpen, onOpenChange, username }: User
310310
return
311311
}
312312

313-
if (userCloseRef.current) {
314-
userCloseRef.current = false
315-
onOpenChange(false)
316-
return
317-
}
313+
// Always allow closing - user should be able to close the dialog
314+
userCloseRef.current = false
315+
onOpenChange(false)
318316
},
319317
[onOpenChange],
320318
)
321319

322-
useEffect(() => {
323-
if (!isOpen) return
324-
325-
const handleCloseButtonClick = (e: MouseEvent) => {
326-
const target = e.target as HTMLElement
327-
if (target.closest('[data-radix-dialog-close]') || target.closest('button[aria-label="Close"]')) {
328-
userCloseRef.current = true
329-
}
330-
}
331-
332-
document.addEventListener('click', handleCloseButtonClick, true)
333-
return () => {
334-
document.removeEventListener('click', handleCloseButtonClick, true)
335-
}
336-
}, [isOpen])
337-
338320
return (
339321
<Dialog open={isOpen} onOpenChange={handleOpenChange}>
340322
<DialogContent

0 commit comments

Comments
 (0)