Skip to content

feat(spec-site): replace all confirm()/alert() with custom modal#23

Merged
AngryJay91 merged 2 commits intomainfrom
fix/issue-13-custom-modal
Mar 31, 2026
Merged

feat(spec-site): replace all confirm()/alert() with custom modal#23
AngryJay91 merged 2 commits intomainfrom
fix/issue-13-custom-modal

Conversation

@AngryJay91
Copy link
Copy Markdown
Owner

Closes #13

Changes

Replaced all native confirm()/alert() calls with custom modal via useConfirm composable across 11 page files:

  • DashboardPage.vue — convertToEpic, convertToStory
  • DocsEditor.vue — save validation/error
  • DocsPage.vue — restoreRev
  • MemosPage.vue — createDocFromMemo, deleteReply, convertToStory, convertToInitiative, createMemo
  • MockupEditorPage.vue — onDelete
  • MockupListPage.vue — deleteMockup
  • MockupViewerPage.vue — copySpec
  • board/BoardAdmin.vue — handleDeleteTask
  • board/StoryDetailPanel.vue — handleMergeOk
  • retro/RetroHeader.vue — nextPhase, handleReset
  • retro/RetroPage.vue — handleExport

SlashCommand.ts was already migrated (confirmed clean).

Pattern used

const { showConfirm, showAlert } = useConfirm()
// confirm → await showConfirm(...)
// alert   → await showAlert(...)

All functions made async where needed.

@AngryJay91 AngryJay91 merged commit 789e45a into main Mar 31, 2026
2 checks passed
@AngryJay91 AngryJay91 deleted the fix/issue-13-custom-modal branch March 31, 2026 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

P2-1: confirm()/alert() → 커스텀 모달 교체

1 participant