UI: Part 2 - hds adoption replace <Modal> #23398
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
github-actions
bot
added
the
hashicorp-contributed-pr
hellobontempo
commented
Sep 28, 2023
| <Modal @title="Copy your generated key" @onClose={{action (mut @isModalActive) false}} @isActive={{@isModalActive}}> | ||
| <section class="modal-card-body"> | ||
| <div class="box is-shadowless is-fullwidth is-sideless"> | ||
| {{#if @isModalActive}} |
There was a problem hiding this comment.
all of the transit modals have similar styling - so just one screenshot :)
hellobontempo
commented
Sep 28, 2023
| @@ -192,16 +192,25 @@ | |||
| </div> | |||
| </div> | |||
| </form> | |||
| <Modal @title="Results" @onClose={{action (mut @isModalActive) false}} @isActive={{@isModalActive}}> | |||
There was a problem hiding this comment.
Previously the modal didn't even have a close button! 🤯
hellobontempo
commented
Sep 28, 2023
| {{#if @isModalActive}} |
hellobontempo
commented
Sep 28, 2023
| @@ -65,7 +65,8 @@ | |||
| "@ember/test-waiters": "^3.0.0", | |||
| "@glimmer/component": "^1.1.2", | |||
| "@glimmer/tracking": "^1.1.2", | |||
| "@hashicorp/ember-flight-icons": "^3.0.9", | |||
| "@hashicorp/design-system-components": "^2.12.2", | |||
There was a problem hiding this comment.
had to upgrade because @container support for <Hds::Copy::Snippet> was added recently
hellobontempo
changed the title
kiannaquach
approved these changes
Sep 28, 2023
|
Build Results: |
hellobontempo
merged commit df93ae8
into
ui/VAULT-17315-hds-adoption-replace-Modal
54 of 56 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.
transit key action modals