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

ACE validation flags an error related to the MathML structure used in display equations #434

Closed
rajas20052001 opened this issue Feb 19, 2025 · 7 comments

Comments

@rajas20052001
Copy link

rajas20052001 commented Feb 19, 2025

We are encountering an issue with the ACE 1.3.6 validation tool when processing display equations captured according to the specification. The error appears when validating MathML content within

.

Error Details:
ACE Version: 1.3.6
Issue: ACE validation flags an error related to the MathML structure used in display equations.
Expected Behavior: The MathML should pass validation as it adheres to thespecification.
Observed Behavior: ACE flags an error, preventing successful validation.

Steps to Reproduce:
Validate the following HTML snippet using ACE 1.3.6.
Observe the reported error in the validation output.

HTML tagging:

(net rate of change of mass within element)=(flux of solute out of the element)-(flux of solute into the element)±(loss or gain of solute mass due to reaction).

CSS:
.disp-formula
{
margin-top: 1em;
margin-bottom: 1em;
margin-left: 0em;
text-align: center;
text-indent: 0em;
overflow-x:auto;
}

@danielweck
Copy link
Member

Hello, what is the error / validation message generated by the evaluation? Are you able to privately share the EPUB? Thank you.

Related: #410 and #398

@rajas20052001
Copy link
Author

rajas20052001 commented Feb 19, 2025

Image

HTML: "<p class="disp-formula"><math xmlns="http://www.w3.org/1998/Math/MathML"><mtable columnalign="right center left"><mtr><mtd><mo>(</mo><mtext>net rate of change of mass within element</mtext><mo>)</mo></mtd><mtd><mo>=</mo></mtd><mtd><mo>(</mo><mtext>flux of solute out of the element</mtext><mo>)</mo></mtd></mtr><mtr><mtd></mtd><mtd><mo>-</mo></mtd><mtd><mo>(</mo><mtext>flux of solute into the element</mtext><mo>)</mo></mtd></mtr><mtr><mtd></mtd><mtd><mo>&#177;</mo></mtd><mtd><mrow><mo>(</mo><mtext>loss or gain of solute mass due to reaction</mtext><mo>)</mo></mrow><mo>.</mo></mtd></mtr></mtable></math></p>"

CSS:
.disp-formula
{
margin-top: 1em;
margin-bottom: 1em;
margin-left: 0em;
text-align: center;
text-indent: 0em;
overflow-x:auto;
}

@mattgarrish
Copy link

FYI, you need to use backticks to escape markup in a github comment.

@danielweck
Copy link
Member

This looks like the kind of bug report that would be really useful to the Deque Axe dev team. I will try to build a minimal reproduction HTML file to test in isolation.

@danielweck
Copy link
Member

see dequelabs/axe-core#3285

@pkra
Copy link
Contributor

pkra commented Feb 19, 2025

This is correctly flagged though. It complains about a scrollable region not being focusable. The p is set to overflow-x and is not focusable. The contents of the p easily cause overflow.

@danielweck
Copy link
Member

ah, well spotted Peter! Thank you :)

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