Skip to content

Conversation

@segovia
Copy link
Contributor

@segovia segovia commented Oct 26, 2022

This fixes an issue where aria-labels for specific elements might be contained in a shadow dom

Copy link
Collaborator

@olivierwilkinson olivierwilkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

I had a look at the v8 release for DTL and it looks like all the breaking changes are for APIs this library doesn't expose so I'll release this as a feature instead of a breaking change. The waitFor timeout and interval stuff has an impact on findBy queries but it seems like it only matters if the global clock is mocked out by Jest which we don't need to worry about.

Thanks for raising the PR 😄

@olivierwilkinson olivierwilkinson merged commit cb6993e into testing-library:main Oct 27, 2022
@github-actions
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants