Problem/Motivation

The Webform module does not provide a consistent UX when deleting entities, plugins, and elements.

@see #2773205: Come up with a design for highly destructive operations in confirm forms

Proposed resolution

Provide a consistent UX for all confirm/delete actions

Remaining tasks

  • Audit existing delete/confirm forms
  • Move all delete/confirm forms to modal dialogs
  • Create WebformConfirmFormBase
  • Create WebformEntityDeleteForm
  • Write basic test coverage
  • Update existing forms

User interface changes

  • Alway opening delete actions in a modal dialog
  • Display an info/warning status describing what is being deleted
  • For destructive behaviors provide a confirmation checkbox
  • Clearly, state "What is being deleted?" (ie Configuration, content, translations, and relationships)

API changes

  • New WebformConfirmFormBase
  • New WebformEntityDeleteForm

Data model changes

N/A

Form

Entity Delete Forms

\Drupal\webform\WebformEntityDeleteForm
/admin/structure/webform/manage/contact/delete

\Drupal\webform\Form\WebformSubmissionDeleteForm
/admin/structure/webform/manage/WEBFORM_ID/submission/SID/delete

\Drupal\webform\WebformOptionsDeleteForm
/admin/structure/webform/config/options/WEBFORM_OPTION_ID/delete

\Drupal\webform_image_select\WebformImageSelectImagesDeleteForm
/admin/structure/webform/config/images/WEBFORM_IMAGES_ID/delete

\Drupal\webform_access\WebformAccessGroupDeleteForm
/admin/structure/webform/access/group/ACCESS_GROUP_ID/delete

\Drupal\webform_access\WebformAccessTypeDeleteForm
/admin/structure/webform/access/type/ACCESS_TYPE_ID/delete

Plugin/Element Delete Forms

\Drupal\webform_ui\Form\WebformUiElementDeleteForm
/admin/structure/webform/manage/WEBFORM_ID/element/ELEMENT_KEY/delete

\Drupal\webform\Form\WebformHandlerDeleteForm
/admin/structure/webform/manage/WEBFORM_ID/handlers/HANDLER_KEY/delete

\Drupal\webform\Form\WebformSubmissionDeleteMultiple
\Drupal\webform\Form\WebformSubmissionsDeleteFormBase
/admin/structure/webform/submissions/purge
/admin/structure/webform/manage/WEBFORM_ID/results/clear

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jrockowitz created an issue. See original summary.

jrockowitz’s picture

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 60c383e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed a697ce3 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed 681932e on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

  • jrockowitz committed b1bf480 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
jrockowitz’s picture

Status: Active » Needs review
FileSize
45.73 KB

  • jrockowitz committed b722faa on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...

Status: Needs review » Needs work

The last submitted patch, 15: 3008614-14.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jrockowitz’s picture

Status: Needs work » Needs review
FileSize
46.58 KB

Status: Needs review » Needs work

The last submitted patch, 18: 3008614-18.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jrockowitz’s picture

Status: Needs work » Needs review
FileSize
48.6 KB

  • jrockowitz committed 078773d on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed 1677835 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed 65f1d17 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed c2ee470 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
  • jrockowitz committed f90fee2 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
jrockowitz’s picture

Here is the change record.

  • jrockowitz committed 4ed7904 on webform-3008614-confirm-ux
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
jrockowitz’s picture

  • jrockowitz committed 6457fe2 on 8.x-5.x
    Issue #3008614 by jrockowitz: [meta] Refactor and improve Webform delete...
jrockowitz’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.