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

mglyph elements too small #763

Closed
pkra opened this issue Mar 11, 2014 · 7 comments

Comments

Projects
None yet
2 participants
@dpvc

This comment has been minimized.

Copy link
Member

commented Mar 11, 2014

These examples used unit-less values for the width and height. My recollection is that the original MathML3 draft said that that meant these values were in pixels, and that is how MathJax interprets these (there is explicit code to do that). I could be wrong, but that is how it looks. The usual meaning for unit-less values is as a multiple of the default size, and that seems to be the current interpretation of the spec. I will need to change the code to accommodate that.

@pkra

This comment has been minimized.

Copy link
Member Author

commented Mar 11, 2014

Do you want me to ask for clarification on the MathWG mailing list? This might all be correct, I suppose.

@pkra pkra added the Investigate label Mar 11, 2014

@pkra

This comment has been minimized.

Copy link
Member Author

commented Mar 12, 2014

Hm... on re-running the tests today, they work on Chrome suddenly. (But the problem persists on FF). Very weird.

@pkra

This comment has been minimized.

Copy link
Member Author

commented Mar 12, 2014

As per David Carlisle:

unitless values are interpreted as a factor of the default size

@dpvc

This comment has been minimized.

Copy link
Member

commented Mar 12, 2014

Yes, I mentioned that in my message above. This is clear in the spec now, but my recollection is that the usual behavior was overridden in an earlier draft (the one in place when I added the glyph support).

@dpvc dpvc added Accepted and removed Investigate labels Mar 19, 2014

@dpvc dpvc added this to the Bugfix Version milestone Mar 19, 2014

dpvc pushed a commit to dpvc/MathJax that referenced this issue Mar 19, 2014

Davide P. Cervone
Fix unitless width and height in HTML-CSS mglyph, and fix aspect rati…
…o when only one is specified in SVG output. Resolves issue mathjax#763.

@dpvc dpvc added Merged and removed Ready for Review labels Mar 20, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Mar 20, 2014

=>Merged.

@dpvc dpvc closed this Mar 20, 2014

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

@dpvc

This comment has been minimized.

Copy link
Member

commented Apr 20, 2014

=> In Test Suite

MathMLToDisplay/Presentation/TokenElements/mglyph/issue763.html

dpvc pushed a commit to dpvc/MathJax that referenced this issue Apr 21, 2014

@dpvc dpvc added v2.4 and removed Merged labels Jun 30, 2014

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.