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

[Math Processing Error] with empty table as script in SVG output #392

Closed
fred-wang opened this issue Feb 11, 2013 · 5 comments

Comments

Projects
None yet
2 participants
@fred-wang
Copy link
Contributor

commented Feb 11, 2013

testcase:

<msub>
<mi>x</mi>
<mtable/>
  </msub>

(and the variants with msup, msubsup). This does not happen with e.g an empty mrow or an empty menclose. SVGnotEmpty does not seem to do anything specific for mtable.

TypeError: this.data[this.sub].SVGdata is undefined
file: http://cdn.mathjax.org/mathjax/latest/unpacked/jax/output/SVG/jax.js
line: 1832

@ghost ghost assigned fred-wang Feb 11, 2013

@fred-wang

This comment has been minimized.

Copy link
Contributor Author

commented Feb 11, 2013

So the issue here is the same as in #365 : when an emtpy mtable is created it should save the SVG data.

@dpvc

This comment has been minimized.

Copy link
Member

commented Feb 11, 2013

The patch certainly should fix this problem. I'm wondering, however, if returning at this point is actually a mistake. For example, if the table specifies a frame, but no rows, should there still be a frame rectangle with width and height equal to twice the frame spacing? There are probably other styles and such that could cause non-empty output even when the table is empty.

@fred-wang

This comment has been minimized.

Copy link
Contributor Author

commented Feb 12, 2013

Yes, I guess you are probably right. However, I think that this case is not too important and can be handled in a separate issue. For the moment, I'm really interested in fixing the math processing errors (MathJax should certainly not crash with random MathML code, even more when it's valid, and moreover making MathJax more robust will help the testcase reduction tool).

@dpvc

This comment has been minimized.

Copy link
Member

commented Feb 12, 2013

OK, in that case, I'm marking this as "Ready for Release" and will merge it into deveop.

dpvc pushed a commit to dpvc/MathJax that referenced this issue Feb 12, 2013

@fred-wang

This comment has been minimized.

Copy link
Contributor Author

commented Feb 13, 2013

Crashtests/issue392.html

=> In testsuite

@dpvc dpvc closed this May 17, 2013

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.