Replace browser confirm() with styled confirmation modal #18

New Issue

kyle · 2026-02-08T06:44:20-08:00

kyle commented

2026-02-08 06:44:20 -08:00

Context: Inline window.confirm() is used for delete confirmations across all list pages. These are unstyled browser dialogs that break the polished UI.

Files affected:

frontend/src/pages/VipList.tsx (line 216)
frontend/src/pages/DriverList.tsx (line 215)
frontend/src/pages/EventList.tsx (line 112)
frontend/src/pages/UserList.tsx (lines 97, 103, 110)

Suggested fix:

Create a <ConfirmModal> component (or use Shadcn AlertDialog)
Add useConfirm() hook that returns a promise-based confirm function
Replace all if (confirm(...)) calls with the styled modal

Effort: Small

**Context:** Inline `window.confirm()` is used for delete confirmations across all list pages. These are unstyled browser dialogs that break the polished UI. **Files affected:** - `frontend/src/pages/VipList.tsx` (line 216) - `frontend/src/pages/DriverList.tsx` (line 215) - `frontend/src/pages/EventList.tsx` (line 112) - `frontend/src/pages/UserList.tsx` (lines 97, 103, 110) **Suggested fix:** - Create a `<ConfirmModal>` component (or use Shadcn AlertDialog) - Add `useConfirm()` hook that returns a promise-based confirm function - Replace all `if (confirm(...))` calls with the styled modal **Effort:** Small

kyle referenced this issue

2026-02-08 06:44:30 -08:00

Code Efficiency #8

kyle referenced this issue from a commit

2026-02-08 07:07:27 -08:00

refactor: code efficiency improvements (Issues #9-13, #15, #17-20)

kyle commented

2026-02-08 07:09:58 -08:00

Created with dark-themed styling. Replaced all in VipList, DriverList, EventList, and UserList.

kyle closed this issue

2026-02-08 07:09:58 -08:00

Sign in to join this conversation.