Minor improvements to the shared confirm danger form
Background
The following discussion from !76417 (merged) should be addressed:
-
@timnoah started a discussion: (+2 comments) 👋 @ekigbo thanks for sharing this with UX for a review I just had a quick look and have a few questions:- I do not think the
Confirmation required
help text is necessary since theConfirm
button is already in the disabled state when the input is empty. What are your thoughts on removing the help text? - The
current forks will keep their visibility level
note is in a<ul><li>
list on the original modal. Could this be because there may be more than one? If so it may be better to keep the<ul><li>
format to help structure the content. - Lastly do we typically in existing confirmation modals remove the cancel button? I've always expected to see one
🤔 I remember a discussion where the team opted to keep a form of cancel button available for users in a particular modal, but please disregard if this conversation has already happened in your stage😅
cc @fneill for Technical Writing awareness
- I do not think the
Implementation Guide
-
Hide confirmation required
error state of the form - I do not think theConfirmation required
help text is necessary since theConfirm
button is already in the disabled state when the input is empty. What are your thoughts on removing the help text? -
Add a cancel button - Lastly do we typically in existing confirmation modals remove the cancel button? I've always expected to see one 🤔 I remember a discussion where the team opted to keep a form of cancel button available for users in a particular modal, but please disregard if this conversation has already happened in your stage😓 -
Update storybook stories + jest specs
Edited by Ezekiel Kigbo