feat(a11y): add scripts for reporting a11y violations #517
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
seanforyou23
requested review from
andresgalante,
dgutride,
dmiller9911 and
jgiardino
|
Deploy preview for pf-next ready! Built with commit d50fc4f |
dgutride
approved these changes
Aug 6, 2018
add sitemap generator script stop tracking generated files add a test/audit runner add basic reporters add script for running as part of ci
seanforyou23
force-pushed
the
axe-tooling
branch
from
a32f1ed
to
d50fc4f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a few new scripts to help with identifying accessibility issues with our components. These scripts handle creating a sitemap file, which is processed by the aXe accessibility engine, which then outputs a report that can be used to guide developers on how to solve the issues being reported.
This introduces a nominal number of new dependencies, which are:
How to use:
npm run devnpm run a11yWe'll soon want the ability for this to run as part of a CI job, this can be completed as part of follow up work. For now, the main thing is getting this tool into our hands so we can begin chipping away at the existing problems we have.
Should help close #466
add sitemap generator script
don't track generated files
add an audit runner
add basic reporters
add script for running as part of ci