From 2480049fd1f67c44acf2bc1c2f9268bab26a3050 Mon Sep 17 00:00:00 2001
From: Mo Tarbin <mhed.t91@gmail.com>
Date: Sun, 16 Aug 2026 01:26:28 -0400
Subject: [PATCH] Improve user feedback and modal behavior in ErrorReportModal
 and PolicyUpdateModal

---
 src/views/Modals/ErrorReportModal.jsx  |  2 +-
 src/views/Modals/PolicyUpdateModal.jsx | 14 ++++++++++----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/views/Modals/ErrorReportModal.jsx b/src/views/Modals/ErrorReportModal.jsx
index be4d41b..5c8333e 100644
--- a/src/views/Modals/ErrorReportModal.jsx
+++ b/src/views/Modals/ErrorReportModal.jsx
@@ -329,7 +329,7 @@ const ErrorReportModal = ({ error, errorInfo, onClose, open }) => {
               level='body-sm'
               sx={{ color: 'text.secondary', mt: 0.5 }}
             >
-              Thanks — this goes straight to the people who can fix it.
+              Thanks! this goes straight to the people who can fix it.
             </Typography>
           </Box>
 
diff --git a/src/views/Modals/PolicyUpdateModal.jsx b/src/views/Modals/PolicyUpdateModal.jsx
index d16717d..de22db7 100644
--- a/src/views/Modals/PolicyUpdateModal.jsx
+++ b/src/views/Modals/PolicyUpdateModal.jsx
@@ -27,13 +27,16 @@ const PolicyUpdateModal = ({ onAcknowledge, onClose, open }) => {
   const { t } = useTranslation()
   const { ResponsiveModal } = useResponsiveModal()
 
-  const handleClose = () => {
+  // Acknowledgement is the only way out: the backdrop, escape key, and close
+  // button are all disabled so the user must press "Got it".
+  const handleAcknowledge = () => {
     onAcknowledge?.()
     onClose()
   }
 
+  // Reading a document must not dismiss the notice; the modal is still waiting
+  // on an acknowledgement when the user returns from the browser.
   const openDocument = path => {
-    handleClose()
     openUrl(`${POLICY_BASE_URL}${path}`)
   }
 
@@ -47,7 +50,10 @@ const PolicyUpdateModal = ({ onAcknowledge, onClose, open }) => {
   return (
     <ResponsiveModal
       open={open}
-      onClose={handleClose}
+      onClose={handleAcknowledge}
+      closeOnBackdrop={false}
+      closeOnEscape={false}
+      showCloseButton={false}
       size='sm'
       title={t('policyUpdate.title')}
       description={t('policyUpdate.subtitle')}
@@ -55,7 +61,7 @@ const PolicyUpdateModal = ({ onAcknowledge, onClose, open }) => {
         <ModalActions
           primary={{
             label: t('policyUpdate.acknowledge'),
-            onClick: handleClose,
+            onClick: handleAcknowledge,
             sx: { width: { xs: '100%', sm: 'auto' } },
           }}
         />