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.

Sign up for GitHub

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

Jump to bottom

Adding test cases for label tests #709

Merged
merged 1 commit into from
Jun 21, 2020
Merged

Adding test cases for label tests #709

merged 1 commit into from
Jun 21, 2020

Conversation

jessebeach
Copy link
Collaborator

@jessebeach jessebeach commented Jun 20, 2020 •
edited by ljharb
Loading

Add test cases for:

  1. label-has-associated-control
  2. control-has-associated-label

Closes #691.

@jessebeach jessebeach linked an issue Jun 20, 2020 that may be closed by this pull request
jsx-a11y/control-has-associated-label issuing warnings for hidden fields #691
Closed
@jessebeach jessebeach linked an issue Jun 21, 2020 that may be closed by this pull request
control-has-associated-label returns false positive, even wrapped AND with htmlFor #681
Closed
@jessebeach jessebeach merged commit b6ffb6b into master Jun 21, 2020
@jessebeach jessebeach deleted the 691-fix branch June 21, 2020 06:47
This was referenced May 24, 2024
Open
Open
@richyhoopd richyhoopd mentioned this pull request Jun 22, 2024
Open
@Luis-Kristen Luis-Kristen mentioned this pull request Jun 24, 2024
Open
@ManuelDevWeb ManuelDevWeb mentioned this pull request Jun 27, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ljharb ljharb ljharb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@jessebeach @ljharb