bootlint icon indicating copy to clipboard operation
bootlint copied to clipboard

Provide additional linters to check for accessibility features

Open foosbar opened this issue 9 years ago • 6 comments

Bootstrap does a nice job documenting the places where aria-* and role attributes should be used. I'd also like to validate those with bootlint. I've been working on implementing these as an extension to bootlint, but if there's value in adding them directly into bootlint, I'd be happy to share.

foosbar avatar Aug 10 '15 20:08 foosbar

Even though I'm fully behind the idea of automatically doing this, I'm not sure bootlint itself is a good candidate to automate those sorts of checks.

The main purpose of bootlint is to root out common mistakes in markup structure, which won't be affected by ARIA.

There are other tools out there that do automated accessibility checking, including some open source ones that play nicely with Grunt, such as arialinter.

zacechola avatar Aug 12 '15 01:08 zacechola

Would love a tool to track our the quality of our accessibility features.

mdo avatar Aug 12 '15 04:08 mdo

It may not be a general accessibility check like arialinter, but making it part of bootlint would make sense since certain aria attributes are associated with the bootstrap class semantics.

foosbar avatar Aug 12 '15 14:08 foosbar

I'm not opposed in principle, so long as it's Bootstrap-specific. Though Bootstrap v4 will be coming relatively soon, and one of the goals there is to add some JavaScript that will add many (though probably not all) of the relevant aria-* & role attributes automatically, making this less of an issue.

cvrebert avatar Aug 12 '15 19:08 cvrebert

I committed some changes to my branch. Would love some feedback on the direction, but it's not ready for PR yet as some of the existing tests are broken.

I am thinking of either making the accessibility checks warnings or making it an optional check.

foosbar avatar Aug 17 '15 13:08 foosbar

I love that Bootstrap 4 will add those in automatically. This will let us validate some of the localized attributes like aria-label.

foosbar avatar Aug 17 '15 13:08 foosbar