From ad5a0e1e415e65cf3f5cb56ec774b22ba417511a Mon Sep 17 00:00:00 2001 From: mavis Date: Thu, 30 Jul 2026 11:13:54 +0000 Subject: [PATCH] fix: replaced alert calls with toast notifications in RoutineOverviewModal --- .../src/components/Routine/RoutineOverviewModal.jsx | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/frontend/src/components/Routine/RoutineOverviewModal.jsx b/frontend/src/components/Routine/RoutineOverviewModal.jsx index fc87c4e8..7bce61ea 100644 --- a/frontend/src/components/Routine/RoutineOverviewModal.jsx +++ b/frontend/src/components/Routine/RoutineOverviewModal.jsx @@ -15,6 +15,7 @@ export default function RoutineOverviewModal({ const [isExporting, setIsExporting] = useState(false); const [toastMessage, setToastMessage] = useState(""); const [showToast, setShowToast] = useState(false); + const [toastIsError, setToastIsError] = useState(false); useEffect(() => { document.body.style.overflow = "hidden"; @@ -23,8 +24,9 @@ export default function RoutineOverviewModal({ }; }, []); - const triggerToast = (msg) => { + const triggerToast = (msg, isError = false) => { setToastMessage(msg); + setToastIsError(isError); setShowToast(true); setTimeout(() => { setShowToast(false); @@ -40,7 +42,7 @@ export default function RoutineOverviewModal({ triggerToast("Share link copied!"); } catch (err) { console.error(err); - alert("Failed to copy share link"); + triggerToast("Failed to copy share link", true); } }; @@ -53,7 +55,7 @@ export default function RoutineOverviewModal({ triggerToast("Routine summary copied!"); } catch (err) { console.error(err); - alert("Failed to copy summary"); + triggerToast("Failed to copy summary", true); } }; @@ -66,7 +68,7 @@ export default function RoutineOverviewModal({ triggerToast("PDF generated!"); } catch (err) { console.error(err); - alert("Failed to export PDF"); + triggerToast("Failed to export PDF", true); } finally { setIsExporting(false); } @@ -89,7 +91,7 @@ export default function RoutineOverviewModal({
-
+

{toastMessage}