Skip to content

fix : replace alert calls with toast notifications in RoutineOverviewModal #1986

Description

@tmdeveloper007

Summary of What Needs to be Done

Replace all remaining browser alert() calls in frontend/src/components/Routine/RoutineOverviewModal.jsx with the existing inline toast notification system (triggerToast function) for a non-blocking user experience.

Changes that Need to be Made

The file already has toast infrastructure (triggerToast function, showToast state, toastMessage). Replace these 3 alert calls:

  • alert("Failed to copy share link") -> triggerToast("Failed to copy share link")
  • alert("Failed to copy summary") -> triggerToast("Failed to copy summary")
  • alert("Failed to export PDF") -> triggerToast("Failed to export PDF")

Impact that it would Provide

  • Consistent UX across the routine management feature
  • Eliminates blocking alert dialogs that interrupt user workflow
  • Better UX on mobile where alert dialogs are especially disruptive

Note: Please assign this issue to the tmdeveloper007 account.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingfeatureImprove existing feature or add newfrontendIssues related to UI/UX

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions