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

Vertical stretching arrows in table cells can cause extra space between rows #468

Closed
dpvc opened this issue May 17, 2013 · 5 comments
Closed
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Available v2.2

Comments

@dpvc
Copy link
Member

dpvc commented May 17, 2013

This example shows too much space between the A and the arrow:

<math display="block">
<mtable>
<mtr><mtd><mi>A</mi></mtd></mtr>
<mtr><mtd><mo stretchy="true" minsize="2em">&#x2193;</mo></mtd></mtr>
</mtable>
</math>

It turns out that the mtable.js code has a wrong index when recording the height and depth of the row when a cell contains a single stretchy (embellished) operator, so it sets the height and depth of the wrong row (in this case, the top row rather than the bottom row).

dpvc added a commit to dpvc/MathJax that referenced this issue May 21, 2013
@dpvc
Copy link
Member Author

dpvc commented May 21, 2013

The issue468 branch of my fork of MathJax contains the fix.

dpvc added a commit that referenced this issue May 22, 2013
@dpvc
Copy link
Member Author

dpvc commented May 22, 2013

=> Merged

@fred-wang
Copy link
Contributor

MathMLToDisplay/issue468.html

=> In Testsuite

@pkra
Copy link
Contributor

pkra commented May 24, 2013

Can be closed now that it's pushed everywhere?

@dpvc
Copy link
Member Author

dpvc commented May 24, 2013

I had left it open because Fred had it marked to create a test for it. Now that that's done, we can close it.

@dpvc dpvc closed this as completed May 24, 2013
dpvc added a commit to dpvc/MathJax that referenced this issue Feb 14, 2014
…), and make bounding box information correct for stretched character. Resolves issue mathjax#619.
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 Available v2.2
Projects
None yet
Development

No branches or pull requests

3 participants