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

Preserving RDFa, microdata etc on MathML source #860

Closed
pkra opened this issue Jul 8, 2014 · 9 comments

Comments

Projects
None yet
2 participants
@pkra
Copy link
Member

commented Jul 8, 2014

A feature request via SO.

We should preserve microdata, RDFa etc. from the MathML source. In particular now that search engines are running more and more JavaScript.

@dpvc dpvc added Accepted labels Jul 18, 2014

@pkra pkra added this to the Next Release milestone Aug 18, 2014

@pkra

This comment has been minimized.

Copy link
Member Author

commented Aug 18, 2014

This could extend to annotations for speech text.

@pkra

This comment has been minimized.

Copy link
Member Author

commented Aug 18, 2014

Might also help provide #546

@pkra

This comment has been minimized.

Copy link
Member Author

commented Aug 20, 2014

  • everything that's not a MathML attribute should not be trimmed
  • author config can specify exceptions

dpvc pushed a commit to dpvc/MathJax that referenced this issue Sep 14, 2014

Davide P. Cervone
Preserve RDFa and other non-standard attributes from MathML elements …
…in the HTML-CSS or SVG output. Resolves issue mathjax#860, and also handles mathjax#502 as a side-effect.
@dpvc

This comment has been minimized.

Copy link
Member

commented Sep 14, 2014

The issue860 branch of my fork of MathJax preserve this type of data in the HTML-CSS and SVG output (it was always included in the NativeMML output). There is a new configuration parameter that controls what is (and isn't) included, that will need to be documented.

@pkra

This comment has been minimized.

Copy link
Member Author

commented Oct 22, 2014

RE test. Would it be enough to start with a single MathML sample with a selection of attributes (rdfa, microdata, data-*, aria) and then checking that the attributes exist in the output? That would also tackle #502.

@dpvc

This comment has been minimized.

Copy link
Member

commented Oct 22, 2014

There is also a new ignoreMMLattributes configuration parameter, so that needs to be tested as well. One also should check that the alt text makes it to the aria-label if the math element doesn't include one already. Finally, you should test both the main math element and some child element, since the data are to be preserved for any node.

dpvc pushed a commit to dpvc/MathJax that referenced this issue Nov 2, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Nov 13, 2014

==> Merged (Nov. 2)

@dpvc dpvc added Merged and removed Ready for Review labels Nov 13, 2014

@dpvc dpvc closed this Nov 13, 2014

@pkra pkra referenced this issue Jan 14, 2015

Closed

2.5 release work #78

16 of 16 tasks complete

pkra added a commit to mathjax/MathJax-docs that referenced this issue Jan 14, 2015

@pkra

This comment has been minimized.

Copy link
Member Author

commented Jan 15, 2015

==> in documentation mathjax/MathJax-docs#78

@pkra pkra assigned pkra and unassigned pkra Jan 15, 2015

pkra added a commit to mathjax/MathJax-test that referenced this issue Jan 20, 2015

add tests for mathjax/MathJax#860: test that many attributes are now …
…preserved; test ignoreMMLattributes option

dpvc pushed a commit to mathjax/MathJax-test that referenced this issue Jan 20, 2015

@dpvc dpvc added Test Available and removed Test Needed labels Jan 20, 2015

@dpvc

This comment has been minimized.

Copy link
Member

commented Jan 20, 2015

==> In Testsuite

MathMLToDisplay/issue860-1.html
MathMLToDisplay/issue860-2.html

@dpvc dpvc added Fixed v2.5 and removed Merged labels Jan 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.