Skip to content
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

fix: use isVisible helper to exclude hidden elements #1

Closed

Conversation

ph-fritsche
Copy link

Hidden elements should be excluded from tab order.
Hidden attribute can be overwritten by stylesheets.

There is a bug in jsdom that has to be resolved first.
See jsdom/jsdom#3111

Hidden elements should be excluded from tab order.
Hidden attribute can be overwritten by stylesheets.

There is a bug in jsdom that has to be resolved first.
See jsdom/jsdom#3111
benadamstyles and others added 12 commits February 13, 2021 12:44
Trigger click event for userEvent.type('{enter}') on HTMLAnchorElements.

Co-authored-by: Philipp Fritsche <ph.fritsche@gmail.com>
* docs: update README.md

* docs: update .all-contributorsrc

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Philipp Fritsche <ph.fritsche@gmail.com>
* docs: update README.md

* docs: update .all-contributorsrc

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
…brary#567)

* fix(click): focus closest focusable

* refactor: remove redundant check
* docs: update README.md

* docs: update .all-contributorsrc

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
* fix(upload): apply accept attribute (testing-library#558)

* fix(upload): make accept filter opt-in

* docs(upload): add options.applyAccept

* fix(typing): add options.applyAccept

Co-authored-by: Laura Beatris <48022589+LauraBeatris@users.noreply.github.com>
* docs: update README.md

* docs: update .all-contributorsrc

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
* docs: update README.md

* docs: update .all-contributorsrc

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@ph-fritsche
Copy link
Author

ph-fritsche commented Mar 8, 2021

Finally jsdom@16.5.0 is released with the fix.
This can be merged now :)

ph-fritsche and others added 4 commits March 9, 2021 12:21
* docs: update README.md

* docs: update .all-contributorsrc

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@ph-fritsche ph-fritsche deleted the pr-556 branch March 14, 2021 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants