-
Notifications
You must be signed in to change notification settings - Fork 402
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
Invalid MathML markup not throwing an error #835
Comments
I tried to reproduce it several times and every time there was an error, hence all works as expected. |
Argh, I can't reproduce this in an isolated minimal example and of course I fixed the sample where I ran into this and don't recall which of the expressions caused the issue. Sigh. Sorry for the noise. I'll re-open if I run into this again. |
Got it. The problem is inside epub-switch elements. Something like <?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml" xmlns:epub="http://www.idpf.org/2007/ops">
<head>
<title>nested math tags should be invalid</title>
</head>
<body>
<h1>nested math tags should be invalid even inside epub switch</h1>
<epub:switch>
<epub:case required-namespace="http://www.w3.org/1998/Math/MathML">
<math alttext="placeholder" display="block" overflow="scroll" xmlns="http://www.w3.org/1998/Math/MathML">
<math alttext="a"><mn>11</mn>
<mo>,</mo>
<mo>−</mo>
<mn>3</mn>
<mo>,</mo>
<mn>5</mn>
</math>
</math>
</epub:case>
<epub:default>
<img src="../Images/Image0001.svg" alt=""/></epub:default>
</epub:switch>
</body>
</html> With 4.0.2, this gets validated without errors. Sample nested-math-tags.zip (change extension to epub). |
I didn't test master so this may be fixed already. |
Tested against epubcheck 4.0.3 built from master. No errors. |
Done in #934 |
Nested
math
tags should throw an error as they are not valid.The text was updated successfully, but these errors were encountered: