Skip to content

Use danger button variant in comment delete modal

Tom Quirk requested to merge 344156-use-danger-variant-comment-delete-modal into master

What does this MR do and why?

For the Delete Comment modal:

  • Uses danger variant for modal button
  • Updates modal button text

#344156 (closed) (for epic &6854 (closed))

Screenshots or screen recordings

Before After
Screen_Shot_2022-02-09_at_4.38.40_pm Screen_Shot_2022-02-09_at_4.40.33_pm

How to set up and validate locally

  1. Enable feature flag Feature.enable(:bootstrap_confirmation_modals)
  2. Navigate to a Git commit for any project.
  3. Create a comment.
  4. Click the vertical dotdotdot menu and click "delete comment"
  5. Observe. The modal should appear.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #344156 (closed)

Edited by Tom Quirk

Merge request reports