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

Plan to improve accessibility documentation (and testing) #292

Closed
bollwyvl opened this issue Dec 9, 2020 · 5 comments
Closed

Plan to improve accessibility documentation (and testing) #292

bollwyvl opened this issue Dec 9, 2020 · 5 comments

Comments

@bollwyvl
Copy link
Collaborator

bollwyvl commented Dec 9, 2020

Hi folks!

It looks like the lighthouse check (#206) hasn't been taken out 🎉, but I haven't been monitoring closely to find out if it has actually caught any regressions! If so, it might be interesting to write it up in the development section.

More broadly: I've recently been digging into some tools for helping devs think about accessibility concerns while developing, and not waiting until a site is live and someone comes with a genuine issue. It seems for something that's upstream a few levels from where most audiences will actually be trying to read stuff, we can have a big impact here.

Here's a relatively-low-effort incremental plan to getting to a place where it can be iteratively improved.

docs

  • add an accessibility page to the docs suite
    • specifically thinking about accessibility from the sensory/cognitive perspective
      • but could perhaps be expanded to localization (Translations for page elements #257) and privacy (e.g. GDPR), as these constitute things that might exclude audiences (practically/legally)
  • outline what this theme (and documentation site) does, with practical examples, to address these matters:
    • content
    • the HTML/JS/CSS/Python itself
    • continuous integration

testing

  • in CI and (optionally) locally, run an open source accessibility check against the example site
    • since nodejs is sitting around in CI, the go-to appears to be pa11y-ci
      • it does pull in puppeteer (sigh)
      • this works particularly well with a sitemap.xml, and can account for mangling the "production" URL, so...
  • to get the sitemap, use sphinx-sitemap to generate during the build
    • happily, this would help SEO for the actual published docs site :)
    • sadly, this doesn't mean what WCAG means when they say "sitemap", which is more like an index page, and probably a separate matter :(
  • add some config to ignore the initial warnings
    • little preview: hacking this toolchain on a local site based on this theme yields... over 1000!
  • maybe add a nice HTML report to CI

implementation

I'm happy to get the ball rolling, but would ❤️ any other feedback and thoughts!

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Dec 9, 2020

Ha: RTD already generates a sitemap for the production site:

https://pydata-sphinx-theme.readthedocs.io/sitemap.xml

I have no idea how it would interact with sphinx-sitemap, so that would likely need to be disabled when running there in their environment.

@bollwyvl
Copy link
Collaborator Author

PR up: #294

@jpmckinney
Copy link
Contributor

Issue description is that this "builds on #61", but is this not the same as #61? What is the difference?

@trallard
Copy link
Collaborator

trallard commented Feb 8, 2023

Folks looking at this issue - this seems no longer applicable, i.e. this was later implemented in other PRs.
I will be opening other issues for accessibility work so could we close this one to avoid confusion over what has/will be/is currently being done?

@drammock
Copy link
Collaborator

drammock commented Feb 9, 2023

Superceded by #1168

@drammock drammock closed this as completed Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants