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

use-onblur-not-onchange overly strict #11

Closed
erin-doyle opened this issue Apr 4, 2017 · 0 comments
Closed

use-onblur-not-onchange overly strict #11

erin-doyle opened this issue Apr 4, 2017 · 0 comments
Labels

Comments

@erin-doyle
Copy link
Collaborator

I'm getting a use-onblur-not-onchange error for a number of elements that have both onchange and onblur handlers and rightfully so. Looking at the respective rule in the eslint-plugin-jsx-a11y it looks like it is checking for when an element only has onchange it fails otherwise if it has both onchange and onblur it passes. I wonder if this rules needs to be updated to do the same.

@erin-doyle erin-doyle added the bug label Apr 6, 2017
erin-doyle pushed a commit that referenced this issue Apr 27, 2017
Fixed #11 from being too strict testing for use-onblur-not-onchange. …
erin-doyle pushed a commit that referenced this issue Oct 15, 2017
…Now it should pass if onChange is used in addition to onBlur.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant