From 0a18cdd556a33a3b392ed2951508e29632766899 Mon Sep 17 00:00:00 2001 From: diondiondion Date: Wed, 15 Jul 2026 11:28:44 +0200 Subject: [PATCH] [Glitch] Fix page refresh when trying to save custom profile fields Port b40eed5e3c016184784835d522450444edc95d06 to glitch-soc Signed-off-by: Claire --- .../confirmation_modal.tsx | 20 +++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/app/javascript/flavours/glitch/features/ui/components/confirmation_modals/confirmation_modal.tsx b/app/javascript/flavours/glitch/features/ui/components/confirmation_modals/confirmation_modal.tsx index 7d2cde8d3d..a5291656c7 100644 --- a/app/javascript/flavours/glitch/features/ui/components/confirmation_modals/confirmation_modal.tsx +++ b/app/javascript/flavours/glitch/features/ui/components/confirmation_modals/confirmation_modal.tsx @@ -60,13 +60,18 @@ export const ConfirmationModal: React.FC< noCloseOnConfirm = false, noFocusButton = false, }) => { - const handleClick = useCallback(() => { - if (!noCloseOnConfirm) { - onClose(); - } + const handleSubmit = useCallback>( + (e) => { + e.preventDefault(); - void onConfirm(); - }, [onClose, onConfirm, noCloseOnConfirm]); + if (!noCloseOnConfirm) { + onClose(); + } + + void onConfirm(); + }, + [onClose, onConfirm, noCloseOnConfirm], + ); const handleSecondary = useCallback(() => { onClose(); @@ -74,7 +79,7 @@ export const ConfirmationModal: React.FC< }, [onClose, onSecondary]); return ( - + {noFocusButton ? ( @@ -115,7 +120,6 @@ export const ConfirmationModal: React.FC< {/* eslint-disable jsx-a11y/no-autofocus -- we are in a modal and thus autofocusing is justified */}