-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[pickers] Fix usage with Shadow DOM #6952
[pickers] Fix usage with Shadow DOM #6952
Conversation
7d2f8ed
to
14d05f7
Compare
These are the results for the performance tests:
|
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.
I think that if you want to add a test you can do by first creating a container, attaching a Shadow DOM to it, then pass its container.shadowRoot
to the container
option of RTL's render
. You'll need to use custom queries because the default ones won't dive into the internals of the Shadow DOM. https://www.npmjs.com/package/shadow-dom-testing-library may help.
Fixes #6841
Reproduction case before
Reproduction case after
Not sure how to test that
@m4theushw I've also use your utility function for the other occurrence of
document.activeElement