UI: Part 5 - hds adoption replace <Modal> #23471
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
github-actions
bot
added
the
hashicorp-contributed-pr
hellobontempo
commented
Oct 3, 2023
| {{#if @isActive}} |
There was a problem hiding this comment.
disabled confirm
enabled confirm button
There was a problem hiding this comment.
Can this modal appear in both instances you have in the screenshots (with or without the header) or is the second screen what it looked like before?
There was a problem hiding this comment.
The second screenshot was just to show the confirm button styling change 😅
hellobontempo
commented
Oct 3, 2023
|
|
||
| import Component from '@glimmer/component'; | ||
| /** | ||
| * @module ConfirmationModal | ||
| * ConfirmationModal components are used to provide an alternative to ConfirmationButton that automatically prompts the user to fill in confirmation text before they can continue with a potentially destructive action. It is built off the Modal component | ||
| * ConfirmationModal components wrap the <Hds::Modal> component to present a critical (red) type-to-confirm modal. |
There was a problem hiding this comment.
Per design, this modal will always be critical because it's destructive. Previously structure didn't have red modals, which is why we used the warning yellow.
hellobontempo
commented
Oct 3, 2023
| } | ||
|
|
||
| get confirmText() { | ||
| return this.args.confirmText || 'Yes'; |
There was a problem hiding this comment.
Nowhere used the default arg so I removed this to simplify the component
hellobontempo
commented
Oct 3, 2023
| @@ -30,7 +30,6 @@ | |||
| @title="Demote to secondary?" | |||
| @onClose={{action (mut this.isModalActive) false}} | |||
| @isActive={{this.isModalActive}} | |||
| @buttonClass="is-primary" | |||
|
Build Results: |
…T-20470/part-5-modals
zofskeez
approved these changes
Oct 3, 2023
| {{#if @isActive}} |
There was a problem hiding this comment.
Can this modal appear in both instances you have in the screenshots (with or without the header) or is the second screen what it looked like before?
hellobontempo
merged commit 11c8f69
into
ui/VAULT-17315-hds-adoption-replace-Modal
59 of 61 checks passed
hellobontempo
added a commit
that referenced
this pull request
Oct 6, 2023
* UI: Part 1 - hds adoption replace <Modal> (#23363) * replace policy-form modal * replace clients/attribution modal * clients/config modal * scope form odal * remove button type * include toolbar to match other example templates * rotate credentials modal * add toolbar button class for hds buttons * transformation-edit modal * add back test selector * add route arg to button! * update link status * fix link-status tests * remove prevent default * update db tests * update tests * use page alert for hcp link status banner * fix scopy button selector * fix sidebar test * change to neutral banner * UI: Part 2 - hds adoption replace <Modal> (#23398) * upgrade HDS library (adds support for snippet containers * cleanup flight icons * replace transit key action modals * re-add deps as devDeps * remove line * address transit tests * UI: Part 3 - hds adoption replace <Modal> (#23415) * cleanup css * cleanup extra type attr * masked input download modal * use Hds::Button in download button" * fix size of modal * tiny icon fix * refactor download button to always render download icon * update tests * UI: Part 3.5 - hds adoption replace <Modal> (#23448) * replication-promote modal * replication component modals * replication add secondary modal * move update text for diff * UI: Part 4 - hds adoption replace <Modal> (#23451) * k8 configure modal * kv delete modal * ldap modals * pki modals * add trash icon * move deps * UI: Part 5 - hds adoption replace <Modal> (#23471) * replace confirmation modals --------- * UI: Part 6 - hds adoption replace <Modal> (#23484) * search select with modal * policy search select modal * replace date dropdown for client dashboard * change padding to top * update policy example args * lolllll test typo wow * update dropdown tests * shamir flow modals! * add one more container * update test selectors * UI: Final hds adoption replace <Modal> cleanup PR (#23522) * search select with modal * policy search select modal * replace date dropdown for client dashboard * change padding to top * update policy example args * lolllll test typo wow * update dropdown tests * shamir flow modals! * add one more container * update test selectors * remove wormhole and modal component * fix selectors * uninstall wormhole * remove shamir-modal-flow class * fix confirm modal test * fix pki and kv test * fix toolbar selector kv * client and download button test * fix-confirmation-modal-padding * fix replication modal tests so relevant modal opens (#23540) * more confirmation modal tests * adds changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Replaces confirmation modals