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

content-mathml.js and namespaces #1755

Closed
pkra opened this issue May 26, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@pkra
Copy link
Member

commented May 26, 2017

Cf. mathjax/ctop#34

It currently breaks when content has namespace prefixes.

In particular, it cannot be used together with mml3.js

@davidcarlisle

This comment has been minimized.

Copy link

commented May 27, 2017

arguably it's user error to have prefixes if the content is served as tex/html as the pseudo-xml parser used for in text/hml puts everything in the mathml namespace but with "local name" being the name as given rather than the part after the : . Any xmlns:xxx attribute is simply ignored.

that said, it would presumably be easy enough to ignore prefixes when matching the element names in the javascript if you wanted to be kind to the user rather than rigorously follow some quirk in the standards.

I'm not sure how much of my original version still survives here, let me know if you want me to do anything but I would guess any of you are rather better at javascript than me as I'm very much an "occasional user" where javascript is concerned.

@dpvc

This comment has been minimized.

Copy link
Member

commented May 30, 2017

The issue1755 branch of my fork of MathJax resolves this issue.

@dpvc dpvc self-assigned this Jun 29, 2017

@dpvc dpvc modified the milestones: A future release, MathJax v2.7.2 Jul 6, 2017

@dpvc dpvc removed their assignment Jul 7, 2017

dpvc pushed a commit to mathjax/MathJax-test that referenced this issue Jul 10, 2017

@dpvc

This comment has been minimized.

Copy link
Member

commented Jul 10, 2017

==> In Testsuite

MathMLToDisplay/Content/issue1755.html

@dpvc dpvc added Test Available and removed Test Needed labels Jul 10, 2017

dpvc added a commit that referenced this issue Jul 10, 2017

Merge pull request #1756 from dpvc/issue1755
Fix problem with namespaces on internal element.  Resolves #1755.

@dpvc dpvc added Merged and removed Ready for Review labels Jul 10, 2017

@dpvc

This comment has been minimized.

Copy link
Member

commented Jul 10, 2017

==> Merged.

@dpvc dpvc closed this Jul 10, 2017

@dpvc dpvc added Fixed v2.7 and removed Merged labels Jul 18, 2018

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.