Skip to content

SUBMISSION-253: Add delete confirmation modal to Review Files page - #130

Open
DavidLFielding wants to merge 1 commit into
developfrom
SUBMISSION-253-ReviewFileDeleteModal
Open

SUBMISSION-253: Add delete confirmation modal to Review Files page#130
DavidLFielding wants to merge 1 commit into
developfrom
SUBMISSION-253-ReviewFileDeleteModal

Conversation

@DavidLFielding

@DavidLFielding DavidLFielding commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a confirmation dialog to the Review Files page before it deletes files on Continue, for parity with Submit 1.5. Previously the page deleted every checked file (including the ones auto-checked as unused) with no confirmation — unlike the Upload page, which confirms every delete.

Behavior

  • On Continue, if any deletable files are checked, a modal lists them and asks the submitter to confirm before proceeding. Confirm submits; Cancel / Escape / × / backdrop-click dismiss and stay on the page.
  • Continuing with nothing checked proceeds straight through — no dialog.
  • Go Back and other actions are never intercepted.
  • Progressive enhancement: with JavaScript off, the modal never renders and Continue behaves exactly as before, so the no-JS path is unchanged. Confirm submits via requestSubmit(button) so the action=next value is preserved.

Design

Mirrors the 1.5 confirm-on-delete dialog: centered title-case heading with a close (×), a centered "By continuing the following N files will be deleted:" lead, a plain bulleted list, and two equal-width Cancel / Confirm buttons.

One intentional deviation from 1.5: the file list scrolls within its own region rather than scrolling the whole dialog, so a large submission (hundreds of files) keeps the heading and buttons in view.

Tests

  • submit_ce/ui/static/js/tests/review_delete_confirm.test.js — self-contained Node test (DOM stub, no test runner) covering intercept + list/count, Confirm submitting with the button as submitter, Cancel, Escape, and the no-checked-files pass-through. Run with node submit_ce/ui/static/js/tests/review_delete_confirm.test.js.
ReviewFiles-2 0-DeleteModal-Screenshot 2026-08-28 at 11 26 00 AM

@DavidLFielding
DavidLFielding marked this pull request as ready for review August 28, 2026 15:28
@DavidLFielding DavidLFielding self-assigned this Aug 28, 2026
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.

3 participants