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
feat(clipboard): add the ability to specify number of attempts in clipboard directive #17547
feat(clipboard): add the ability to specify number of attempts in clipboard directive #17547
Conversation
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.
Could we add an optional clipboard options provider as part of this? I imagine people would want to set this per-application rather than per-instance.
6d4dbe3
to
30a9da5
Compare
Added an injection token that can be used to configure the number of attempts. |
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.
LGTM
Would be good to document the provider either in this PR or in a follow-up
30a9da5
to
0a307d8
Compare
good to merge after rebasing and fixing the lint error |
0a307d8
to
ff4a67a
Compare
Done @mmalerba. |
It looks like there is an issue with the tests now |
…pboard directive Adds a new input to the `cdkCopyToClipboard` directive which allows consumers to set the number of attempts to try and copy their text. We currently have an example of how to implement attempts in the readme, but this feature makes it more convenient so that consumers don't have to do it on a case-by-case basis.
ff4a67a
to
e158d47
Compare
Looks like I messed up the rebase. It should be good to go now. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Adds a new input to the
cdkCopyToClipboard
directive which allows consumers to set the number of attempts to try and copy their text. We currently have an example of how to implement attempts in the readme, but this feature makes it more convenient so that consumers don't have to do it on a case-by-case basis.