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

Rich content section uses old chemml variable #575

Closed
mattgarrish opened this issue Jan 7, 2025 · 4 comments
Closed

Rich content section uses old chemml variable #575

mattgarrish opened this issue Jan 7, 2025 · 4 comments
Labels
a11y-display-techniques-epub Issue with the epub accessibility metadata display techniques

Comments

@mattgarrish
Copy link
Member

mattgarrish commented Jan 7, 2025

The variable setup still has:

LET chemical_formula_as_chemml be the result of calling check for node on package_document, /package/metadata/meta[@property="schema:accessibilityFeature" and text()="ChemML"].

But in the output checks it now uses a new undefined chemical_formula_as_mathml variable.

It looks like the above variable should now be:

LET chemical_formula_as_mathml be the result of calling check for node on package_document, /package/metadata/meta[@property="schema:accessibilityFeature" and normalize-space()="MathML-chemistry"].

@mattgarrish mattgarrish added the a11y-display-techniques-epub Issue with the epub accessibility metadata display techniques label Jan 7, 2025
@clapierre
Copy link
Collaborator

clapierre commented Jan 7, 2025

I think we need both right? Chemistry could be done in ChemML or done via MathML.

@mattgarrish
Copy link
Member Author

Does anyone put ChemML into EPUB? You can't put it in natively like mathml tags and epub:switch has been deprecated for embedding other grammars.

I guess maybe you could embed a separate ChemML file using an object tag, but then you'd run into it being a foreign resource and needing a fallback.

@mattgarrish
Copy link
Member Author

and needing a fallback.

At least some internal fallback within the object tag, I suppose.

@clapierre
Copy link
Collaborator

ok so for now I will ignore the ChemML feature and use just the MathML-chemistry.
If/when we ChemML makes it into EPUBs we can revisit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-display-techniques-epub Issue with the epub accessibility metadata display techniques
Projects
None yet
Development

No branches or pull requests

2 participants