-
-
Notifications
You must be signed in to change notification settings - Fork 639
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
aria-proptypes support for idlist #454
Comments
This is affecting the airbnb config as well, fwiw. |
I'll get this fixed tonight. 6.2.0 update is on the way :) |
Here's the change in aria-query that is causing the errors: A11yance/aria-query@6978637 What I don't yet understand is why none of the tests in this repo caught this. I don't want to simply update aria-query without getting a test case to fail here first so we don't have something like this happen again in the future. |
[#454] Fix for aria-proptypes rule failure
Closed in #456. |
I have noticed the following errors on Material-UI side trying to upgrade this dependency:
The following aria seem to not be supported as expected:
aria-labelledby
aria-controls
aria-describedby
arial-query returns the type
idlist
. Something that is not supported in the implementation: https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/b800f40a2a69ad48015ae9226fbe879f946757ed/src/rules/aria-proptypes.js#L32The text was updated successfully, but these errors were encountered: