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

avoid-positive-tabindex error #157

Closed
crswll opened this issue Nov 26, 2018 · 2 comments
Closed

avoid-positive-tabindex error #157

crswll opened this issue Nov 26, 2018 · 2 comments
Assignees

Comments

@crswll
Copy link

crswll commented Nov 26, 2018

I was getting an error about avoid-positive-tabindex and many warnings after the renaming of the rules minor patch (which maybe should have been a major, especially due to #145).

I wanted to open this issue as more of a help document than a bug report since figuring out the new names wasn't very simple.

Here goes the adjustments I needed to make in order to get rid of the warnings and fix the error:

Original

'avoid-positive-tabindex': 'warn',
'button-role-space': 'warn',
'hidden-uses-tabindex': 'warn',
'img-uses-alt': 'warn',
'label-uses-for': 'warn',
'mouse-events-map-to-key-events': 'warn',
'no-access-key': 'warn',
'no-hash-ref': 'warn',
'no-unsupported-elements-use-aria': 'warn',
'onclick-uses-role': 'warn',
'onclick-uses-tabindex': 'warn',
'redundant-alt': 'warn',
'tabindex-uses-button': 'warn',
'use-onblur-not-onchange': 'warn',
'valid-aria-role': 'warn',

Updated

'tabindex-no-positive': 'warn',
'click-events-have-key-events': 'warn',
'hidden-uses-tabindex': 'warn',
'img-uses-alt': 'warn',
'label-has-for': 'warn',
'mouse-events-have-key-events': 'warn',
'no-access-key': 'warn',
'no-hash-ref': 'warn',
'aria-unsupported-elements': 'warn',
'onclick-uses-role': 'warn',
'interactive-supports-focus': 'warn',
'img-redundant-alt': 'warn',
'tabindex-uses-button': 'warn',
'no-onchange': 'warn',
'aria-role': 'warn',

Here's a quick diff:
screen shot 2018-11-26 at 11 07 36 am

@erin-doyle
Copy link
Collaborator

Yeah I apologize if this update was not smooth. I was planning to bundle these changes along with some others to make a major but then had a bug fix to put out in the meantime. I thought simply deprecating the old rules with warnings would be a gentle way to inform people of the upgrade path but apparently that was not the experience. I can look into a way to document this better.

@erin-doyle erin-doyle self-assigned this Dec 6, 2018
@erin-doyle
Copy link
Collaborator

I've added a mapping of old to new rules in the README. Hopefully that is more helpful!

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

No branches or pull requests

2 participants