Skip to content

AssistiveMML extension throws error on IE8 [was: HTML-CSS output causes “unknown runtime error” in IE8] #1477

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

Closed
drify opened this issue May 17, 2016 · 4 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Not Needed v2.7

Comments

@drify
Copy link

drify commented May 17, 2016

Just open the test page http://cdn.mathjax.org/mathjax/latest/test/sample-tex.html using IE8. In fact the math renders properly, but the console shows the error.
MathJax 2.5 doesn't have this issue. http://cdn.mathjax.org/mathjax/2.5-latest/test/sample-tex.html

@pkra pkra added the Accepted Issue has been reproduced by MathJax team label May 17, 2016
@pkra
Copy link
Contributor

pkra commented May 17, 2016

Thanks for the report.

The error traces back to https://github.com/mathjax/MathJax/blob/master/unpacked/extensions/AssistiveMML.js#L131 which might be a MathML parsing problem with IE8 -- though I'm a little confused why this wouldn't occur with the CommonHTML as well.

Workaround is to disable the AssitiveMML extension; it's probably useless on IE8 anyway since no AT needs/makes use of it on IE8, I think (only MathPlayer is relevant and that works differently after all).

@pkra pkra added this to the A future release milestone May 17, 2016
dpvc added a commit to dpvc/MathJax that referenced this issue May 17, 2016
…ince there is no reader tht can handle this with IE8 anyway, there isn't much point in trying to figure out whatever bug is the source of this error. Resolves issue mathjax#1477.
@dpvc
Copy link
Member

dpvc commented May 17, 2016

I put a try {...} catch (err) {} around that line, which will prevent the error message, but doesn't solve the problem that is preventing the MathML from being inserted. I don't think it is worth trying to find that, since (as you say), there is no AT that works with it.

The issue1477 branch of my fork of MathJax includes the patch. I think we can include this in 2.x.x, though I don't really have strong feelings either way.

@pkra
Copy link
Contributor

pkra commented May 18, 2016

I don't think it is worth trying to find that, since (as you say), there is no AT that works with it.

👍

I think we can include this in 2.x.x, though I don't really have strong feelings either way.

👍

@pkra pkra changed the title HTML-CSS output causes “unknown runtime error” in IE8 AssistiveMML extension throws error on IE8 [was: HTML-CSS output causes “unknown runtime error” in IE8] May 18, 2016
@dpvc dpvc modified the milestones: MathJax v2.x.x, A future release May 18, 2016
dpvc added a commit that referenced this issue Jun 6, 2016
Catch IE8 errors with inserting MathML from AssistiveMML extension. #1477
@dpvc
Copy link
Member

dpvc commented Jun 6, 2016

==> Merged.

@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Jun 6, 2016
@dpvc dpvc closed this as completed Jun 6, 2016
@dpvc dpvc added v2.7 Fixed and removed Merged Merged into develop branch labels Jan 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Not Needed v2.7
Projects
None yet
Development

No branches or pull requests

3 participants