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

RFC 89 - ATAG 2.0 audit #89

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

thibaudcolas
Copy link
Member

@thibaudcolas thibaudcolas commented Oct 31, 2023

View the audit in HTML. View the audit in standardised ATAG reporting formats on GitHub Gists.

Cite this audit report with a permanent link: wagtail.org/accessibility/atag-audit/


Thank you to the Wagtail accessibility team @wagtail/accessibility for the foundational work that made this audit and report possible. It’s a big deal for us that we finally have this first audit, so THANK YOU @Scotchester @thibaudcolas @jessemenn @kbayliss @Stormheg @SaptakS @anujaraj @janehughes @albinazs @laymonage @jams2. Also thank you to my employer Torchbox for sponsoring this audit and associated reporting.

Additional thanks to @albinazs, @jams2, @laymonage, @thibaudcolas, and @nicklee for their work on Wagtail’s first ever accessibility checker (https://wagtail.org/blog/introducing-wagtails-new-accessibility-checker/), which was essential for us to (partly) meet B.3. and B.4. guidelines.

And finally thanks to people who provided early guidance on conducting ATAG audits in the web-a11y Slack in the #atag channel! This has been an invaluable source of information for us to figure out how to go about this.

- Fail: Image upload fields in the image edit/create form. The image’s title displays as a field next to the visuals. The title acts as alt text by default in Wagtail. This is missing a programmatic association between the title text and image.
- Example: [Wagtail 5.1 - Editing image Boston Cream Pie](https://static-wagtail-v5-1.netlify.app/admin/images/43/)
- Current: The alt text is permanently set to the contents of the Title field on page load.
- Proposed actions:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback from @streats:

Might it be worth adding reference to Chiemezuo’s proposal for improving alt text capabilities?


Proposed actions:

- Document all keyboard commands (SC _A.3.1.6 Present Keyboard Commands_)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback from @streats:

Document all keyboard commands - is this related to 445 Implement Keyboard shortcut documentation for editor in the Wagtail UI #3949? I see that issue moved to a discussion but I’m not clear if the documentation part has been done or not. If they are the same thing, it might be worth linking to it in 427 for context


Proposed actions:

- [Accessibility features documentation #69](https://github.com/wagtail/roadmap/issues/69)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback from @streats:

Accessibility features documentation - the linked issue is closed, so if it’s been completed we could omit this (and if not, maybe re-open that issue?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant