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

Define what the content box is for MathML token elements #139

Closed
bfgeek opened this issue Feb 25, 2019 · 5 comments
Closed

Define what the content box is for MathML token elements #139

bfgeek opened this issue Feb 25, 2019 · 5 comments

Comments

@bfgeek
Copy link

bfgeek commented Feb 25, 2019

Currently the implementations differ here, a very simple test case.
https://www.software.hixie.ch/utilities/js/live-dom-viewer/?saved=6689

I'm not sure what is correct here. However what ever is decided needs a bunch of tests to ensure interoperability.

@fred-wang
Copy link
Contributor

mi and mtext are MathML token elements. Do you mean content box for any MathML element?

issue w3c/mathml#14 might be involved here

@bfgeek
Copy link
Author

bfgeek commented Feb 25, 2019

No - see test case. I think this relates to how the ascender/descender contributes to the block-size of token elements.

@fred-wang fred-wang changed the title Define what the content box is for <mi>/<mtext>/etc elements. Define what the content box is for MathML token elements Feb 25, 2019
@dani31415
Copy link
Contributor

My 5 cents to clarify the issue. On Firefox, the issue gets more clear when comparing an <mi> and a <span> side by side with an outline around it (no margin, no border and no padding):

Side by side mi and span

The red one is the <mi> and the blue is the <span>.

I created an example with mathvariant="normal" (to make the comparison fair).
https://www.software.hixie.ch/utilities/js/live-dom-viewer/?saved=6783

@fred-wang
Copy link
Contributor

The core spec defines what the content box for token elements should be, basically deferring to block layout: https://mathml-refresh.github.io/mathml-core/#text-mtext

@fred-wang
Copy link
Contributor

The layout is described in https://w3c.github.io/mathml-core/#layout-of-mtext

I believe the inconsistency mentioned on this bug is related to the ink ascent/descent thing, so closing in favor of #78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants