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

accessibility feedback on module tables #66

Open
samccann opened this issue Jan 13, 2022 · 3 comments
Open

accessibility feedback on module tables #66

samccann opened this issue Jan 13, 2022 · 3 comments

Comments

@samccann
Copy link
Contributor

From a recent reddit post asking users to evaluate the new tables using the following two links:

https://docs.ansible.com/ansible/devel/collections/community/crypto/acme_certificate_module.html#parameters and https://docs.ansible.com/ansible/devel/collections/community/hashi_vault/hashi_vault_lookup.html#parameters.

I sometimes need to use a screen reader and access these docs using lynx. Here are a few observations :-

When I browse to either link with lynx I get a "bad HTML" message.

It renders anyway but the warning possibly ought to be fixed. I suggest you validate here:- https://validator.w3.org/nu/?doc=https%3A%2F%2Fdocs.ansible.com%2Fansible%2Fdevel%2Fcollections%2Fcommunity%2Fcrypto%2Facme_certificate_module.html against the wc3 standards.

I also tried with links and that didn't display an error.

Next test was actually using the page with a screen reader. It was a bit of a problem because the text structure inside the tables is very confusing when read aloud.

Lastly and this one is really minor, the little 'permalink' symbol you use for linking doesn't have a character in the font on my terminal so it renders as a ? in a box.

When I bring the page up in Firefox and use the screen reader, for some reason the tables don't get read out at all. I get told which cell of the table I am in but not what the content is.

@samccann
Copy link
Contributor Author

These problems existed prior to the new rst based tables so nothing introduced by that feature, fwiw.

@felixfontein
Copy link
Collaborator

The broken HTML (at least the part caused by antsibull, if there are errors left they are probably caused by the theme) should be fixed by ansible-community/antsibull-build#387.

@felixfontein
Copy link
Collaborator

ansible-community/sphinx_ansible_theme#44 and ansible-community/sphinx_ansible_theme#45 fix/improve some of the issues in the theme.

ansible/ansible#76759 bumps antsibull to 0.40.2 for the docs build, and ansible-community/antsibull-build#388 should prevent invalid HTML in the future.

I don't have a screen reader, so I cannot really look at the screen reader issues...

@felixfontein felixfontein transferred this issue from ansible-community/antsibull-build Nov 23, 2022
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

2 participants