-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hds-clipboard
modifier: convert to TypeScript (HDS-3344)
#2120
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
5b1ea6a
to
20a0df6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KristinLBradley I've left a few comments, but before approving I would like @alex-ju to do a check on the types I suggested to use yesterday, it may have not been the right suggestion (or at least, there may be further tweakings to do)
…efined, define signature for modifier and use in default export
…in functions to avoid repetition
Co-authored-by: Alex <alex-ju@users.noreply.github.com> Co-authored-by: Cristiano Rastelli <cristiano.rastelli@hashicorp.com>
6c8f0fc
to
1f673a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@KristinLBradley we did a session this morning with @alex-ju in which we reviewed multiple PRs related to TS and yours too; I hope you won't mind us pushing the commit e47d523 so we could get it approved and merged today |
Now that we test the target is an instance of HTMLElement, there's no need to check the node type
📌 Summary
If merged, this PR converts the
hds-clipboard
modifier to TypeScript.The clipboard modifier is used by:
CopyButton
: convert to TypeScript (HDS-2688) #2100CopySnippet
: convert to TypeScript (HDS-2689) #2110🔗 External links
Jira ticket: HDS-3344
👀 Component checklist
[ ] Percy was checked for any visual regression💬 Please consider using conventional comments when reviewing this PR.