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

Tweak some accessibility conformance claims #660

Merged
merged 4 commits into from
Mar 11, 2024
Merged

Tweak some accessibility conformance claims #660

merged 4 commits into from
Mar 11, 2024

Conversation

robinwhittleton
Copy link
Member

@robinwhittleton robinwhittleton commented Mar 10, 2024

This PR:

  • Updates our accessibility conformance claim to WCAG 2.2
  • Adds a lint check for MathML and describedMath accessibility features where appropriate

It also updates the build process to strip MathML and describedMath accessibility features from the compatible builds, and gets MathML building working again on non-Linux systems.

I guess we’ll want to update books with MathML with these new functionality, but we should wait until we’re building with this code.

Copy link
Member

@acabal acabal left a comment

Choose a reason for hiding this comment

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

Looks good Robin, thanks. Minor comment on geckodriver logic.

se/browser.py Show resolved Hide resolved
New features in WCAG 2.2: https://www.w3.org/TR/WCAG22/#new-features-in-wcag-2-2

- 2.4.11 doesn’t apply to us
- 2.4.12 doesn’t apply to us
- 2.4.13 is based on the ereader’s styles
- 2.5.7 doesn’t apply to us
- 2.5.8 we have an out on, as all of our links are inline
- 3.2.6 doesn’t apply to us
- 3.3.7 doesn’t apply to us
- 3.3.8 doesn’t apply to us
- 3.3.9 doesn’t apply to us
…ilds

We’re converting away from MathML there so it’s no longer accurate.
@acabal
Copy link
Member

acabal commented Mar 10, 2024

Can you fix the mypy errors that came up during the merge checks?

geckodriver was vendored in, but only as an ELF binary which aren’t natively compatible on Macs. This adds ARM and Intel binaries for Mac and a Windows Intel binary too, and falls back to checking the path for other machines.
@acabal acabal merged commit a8850b1 into master Mar 11, 2024
2 checks passed
@acabal
Copy link
Member

acabal commented Mar 11, 2024

Excellent, thanks!

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

Successfully merging this pull request may close these issues.

2 participants