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

Table columns are clipped to column width in HTML-CSS but not SVG #807

Closed
pkra opened this issue May 7, 2014 · 9 comments

Comments

Projects
None yet
2 participants
@pkra
Copy link
Member

commented May 7, 2014

http://www.w3.org/Math/testsuite/build/main/Presentation/TablesAndMatrices/mtable/mtable2-full.xhtml renders quite differently for me.

On Chrome 34, the bottom-center cell overlaps with bottom-left

On FF 29, the bottom-center cell "underlaps", i.e., disappears at the edge of the cell.

@pkra pkra added this to the Next Release milestone Aug 18, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Aug 19, 2014

I don't see any overlapping with the bottom left in SVG output. Can you post a screen shot?

The "underlapping" is actually clipping to the column width. I suppose that could be allowed to overlap the next column to the right. The real issue is that line breaking isn't being done within the columns, but that's really a separate problem.

@pkra

This comment has been minimized.

Copy link
Member Author

commented Aug 19, 2014

I don't see any overlapping with the bottom left in SVG output. Can you post a screen shot?

Here's a screenshot for overlap -- happens on FF 31, Chrome 36, ubuntu 14.04.

screenshot from 2014-08-19 14 11 04

The real issue is that line breaking isn't being done within the columns, but that's really a separate problem.

Right. Should I split this into another issue?

@dpvc

This comment has been minimized.

Copy link
Member

commented Aug 19, 2014

Sorry, I would have called that the bottom-center overlapping the bottom-right not the bottom-left.

So the difference is that the HTML-CSS is clipped and the SVG isn't. I can probably get the HTML-CSS not to clip, which (I suspect) is the better behavior.

Should I split this into another issue?

Line breaking in the columns isn't likely to be done any time soon. You can start an issue if you want just to keep it in mind.

@pkra

This comment has been minimized.

Copy link
Member Author

commented Aug 19, 2014

Ah, sorry for being confused.

So the difference is that the HTML-CSS is clipped and the SVG isn't.

Right. ("clipping" must have eluded me at time of writing...)

I can probably get the HTML-CSS not to clip, which (I suspect) is the better behavior.

To be honest, I dislike both but admittedly without clipping, people don't miss the content. Is overflow: scroll an option?

@dpvc

This comment has been minimized.

Copy link
Member

commented Aug 19, 2014

Is overflow: scroll an option?

Not without some substantial rewriting at the moment. The clipping is actually being done at the column level, not the cell, currently, and I don't think you want the whole column to scroll. Also, scrolling requires block-level elements, and I don't think the cells are stored in block-level elements currently. Finally, I'm not sure I can do that in SVG anyway (at least not without implementing it myself).

@pkra

This comment has been minimized.

Copy link
Member Author

commented Aug 19, 2014

Alright -- ouch on all accounts ;-) no-clipping it is.

@dpvc dpvc changed the title MML test suite mtable2 inconsistencies Firefox vs Chrome Table columns are clipped to column width in HTML-CSS but not SVG Sep 11, 2014

dpvc pushed a commit to dpvc/MathJax that referenced this issue Sep 14, 2014

Davide P. Cervone
Don't clip left or right-hand sides, only top and bottom (which are w…
…hat need clipping to hide exctra height/depth, and prevent the positioning spans from overlaying the material above). Resolves issue mathjax#807, and may hekp issue mathjax#646.
@dpvc

This comment has been minimized.

Copy link
Member

commented Sep 14, 2014

The issue807 branch of my fork of MathJax should resolve this.

@dpvc

This comment has been minimized.

Copy link
Member

commented Nov 13, 2014

==> Merged (Nov. 2)

@dpvc dpvc added Merged and removed Ready for Review labels Nov 13, 2014

@dpvc dpvc closed this Nov 13, 2014

dpvc pushed a commit to mathjax/MathJax-test that referenced this issue Jan 17, 2015

@dpvc dpvc added Test Available and removed Test Needed labels Jan 17, 2015

@dpvc

This comment has been minimized.

Copy link
Member

commented Jan 17, 2015

==> In Testsuite

MathMLToDisplay/Presentation/TablesAndMatrices/issue807.html

@dpvc dpvc added Fixed v2.5 and removed Merged labels Jan 30, 2015

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.