devstygian / devstygian/Resto-POS
Feat: Convert all “localhost says” alerts to modal dialogs
- Lingua principale
- PHP
- Stelle
- 4
- Fork
- 5
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
### 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.).
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia con il componente modal esistente “Something Went Wrong”, quindi analizza l’uso di alert() in notif.js e le azioni degli ordini. Controlla le pagine degli ordini, del menu, delle notifiche e degli account per mappare i flussi interessati. Il lavoro è completato quando non rimangono alert “localhost says”, il modal supporta i tipi di messaggio e i callback di conferma elencati e funziona in modo responsive con accesso da tastiera.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- accessibility, design, frontend
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 48/100