A11y

[lc512k]

This project's in really good shape a11y-wise 👏

I've only added some minor fixes:

• Removed <H6>s from the footer
• pa11y is no Ignoring ads as well as comments
• added context to social media links for screen readers, i.e. "Share on Twitter" rather than "Twitter"
• added "Opens in a new window" to _blank targets
• removed #void from copyright href and replaced with dummy url. Leaving #void in accidentally would cause a wcag violation
• added a11y:local script to run a11y checks locally manually
• made npm a11y script names shorter for a better chance of us typing them locally 😬
• removed warnings and notices from all pa11y output (left just errors)

One thing I would strongly suggest is removing the --threshold argument in https://github.com/ft-interactive/starter-kit/blob/master/package.json#L85-L86 but have left that to you, as these would mean the build fails if there's a single error, similar to the qa tests

This template currently passes pa11y and has 0 errors in HTMLCodeSniffer

[lc512k]

Pleasure! Thank you for having me, @joannaskao!