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

XHTML annotation with MathML descendant content valid or invalid? #1105

Open
mattgarrish opened this issue Mar 6, 2020 · 4 comments
Open
Assignees
Labels
priority: low To be processed when time allows spec: EPUB 3.x Impacting the support of EPUB 3.x specifications status: in discussion The issue is being discussed by the development team type: spec The issue is related to a Specification update

Comments

@mattgarrish
Copy link
Member

mattgarrish commented Mar 6, 2020

Another oddity in the valid tests is the one for MathML inside an XHTML annotation.

Inside the test we still have this comment:

<x:p>This annotation cannot contain math (sch test)</x:p>

as this was an error until last year. It is still flagged as an error by validator.nu.

The MathML spec isn't too specific about the element's use:

Alternate representation annotations are most often used to provide renderings for an expression, or to provide an equivalent representation in another markup language.

While XHTML is another markup language, using presentation mathml again inside it seems to defeat the purpose of it being an alternative to presentation mathml.

I'm wondering if we should at least open this against the validator schemas and see if we can harmonize which interpretation is the right one?

@murata2makoto
Copy link
Contributor

Are you saying that this should be an issue against EPUB 3 rather than epubcheck? I would agree.

@mattgarrish
Copy link
Member Author

mattgarrish commented Mar 6, 2020

Are you saying that this should be an issue against EPUB 3 rather than epubcheck?

Might come down to that.

It looks like an addition to the html schemas. The schema in the MathML spec doesn't restrict the content model for annotations, but the one in the validator github repo adds restricted cases for xhtml, svg and mathml annotations.

Seems like a common-sense addition and we're supposed to follow html validation unless we have a specific override.

@rdeltour rdeltour added spec: EPUB 3.x Impacting the support of EPUB 3.x specifications status: in discussion The issue is being discussed by the development team type: spec The issue is related to a Specification update labels Mar 12, 2020
@rdeltour
Copy link
Member

rdeltour commented Dec 8, 2022

@mattgarrish any new thoughts on that? I'm trying to see if this can be tackled and resolved in v5.0.0.

@rdeltour rdeltour added this to the v5.0.0-rc milestone Dec 8, 2022
@rdeltour rdeltour self-assigned this Dec 15, 2022
@rdeltour
Copy link
Member

I'm moving this out of the v5.0.0 milestone, as it looks like the specs are not definitive about this, and it is low priority. Thoughts still welcome.

@rdeltour rdeltour removed this from the v5.0.0-rc milestone Dec 31, 2022
@rdeltour rdeltour added the priority: low To be processed when time allows label Dec 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low To be processed when time allows spec: EPUB 3.x Impacting the support of EPUB 3.x specifications status: in discussion The issue is being discussed by the development team type: spec The issue is related to a Specification update
Projects
None yet
Development

No branches or pull requests

3 participants