UI: Part 3.5 - hds adoption replace <Modal> #23448
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
|
Build Results: |
hellobontempo
commented
Oct 2, 2023
| {{#if this.isModalActive}} |
hellobontempo
commented
Oct 2, 2023
| {{#if this.isModalActive}} |
hellobontempo
commented
Oct 2, 2023
Comment on lines
-132
to
-142
| toggleModal(successMessage) { | ||
| if (!!successMessage && typeof successMessage === 'string') { | ||
| this.flashMessages.success(successMessage); | ||
| } | ||
| // use copy browser extension to copy token if you close the modal by clicking outside of it. | ||
| const htmlSelectedToken = document.querySelector('#token-textarea'); | ||
| htmlSelectedToken.select(); | ||
| document.execCommand('copy'); | ||
|
|
||
| this.toggleProperty('isModalActive'); | ||
| this.transitionToRoute('mode.secondaries'); |
There was a problem hiding this comment.
this method wasn't working, probably because execCommand was deprecated [docs]
hellobontempo
commented
Oct 2, 2023
| <section class="modal-card-body"> | ||
| <Hds::Modal | ||
| id="replication-copy-token-modal" | ||
| @onClose={{action "closeTokenModal"}} |
There was a problem hiding this comment.
Kept {{action}} helper syntax because these actions are from an outdated controller
hellobontempo
commented
Oct 2, 2023
| </Modal> | ||
| </M.Body> | ||
| <M.Footer> | ||
| <Hds::ButtonSet> |
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.
replication action modals