Skip to content

Feat: Convert all “localhost says” alerts to modal dialogs #59

@devstygian

Description

@devstygian

Description:

Replace all default browser alert dialogs (e.g., alert(), “localhost says”) with a custom modal component to improve user experience and UI consistency across the system.

Goals:

  • Remove all usage of alert() in the codebase.
  • Replace with a reusable modal component (e.g., confirmation, success, error).
  • Ensure consistent styling with the current UI theme.
  • Support different message types:
  • Success
  • Error
  • Warning
  • Confirmation (Yes/No)

Acceptance Criteria:

  1. No “localhost says” alerts appear anywhere in the system.
  2. All alerts are replaced with styled modals.
  3. Modal can be triggered globally via JavaScript.
  4. Confirmation modals support callbacks (e.g., delete actions).
  5. Works across all pages (orders, menu, notifications, accounts).
  6. Responsive and accessible (keyboard + mobile-friendly).

Notes:

  • Can reuse the “Something Went Wrong” modal component as a base.
  • Suggested approach: create a global showModal(type, message, callback) function.
  • Gradually refactor existing JS files (notif.js, order actions, etc.).

Metadata

Metadata

Assignees

No one assigned

    Labels

    UI/UXUser interface and experienceenhancementNew feature or requestfrontendTasks related to UI, dashboard, designhelp wantedExtra attention is needed

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions