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

mspace and CSS #59

Open
fred-wang opened this issue Jan 27, 2020 · 1 comment
Open

mspace and CSS #59

fred-wang opened this issue Jan 27, 2020 · 1 comment

Comments

@fred-wang
Copy link
Contributor

cc @bfgeek @rwlbuis
This is related to CSS issues w3c/mathml#45 w3c/mathml#50 w3c/mathml#49 w3c/mathml#13

Rob has implemented the mspace element by mapping (in C++):

CSS width: width attribute value
CSS vertical-align: height attribute value
CSS height: calc(height attribute value+depth attribute value)

This has various consequences which make it different from the current specs and Gecko/WebKit implementations:

(1) One can now specify these CSS property directly on the mspace element and this will change its rendering and override the attribute values. The spec instead says they are ignored: https://mathml-refresh.github.io/mathml-core/#css-styling

(2) For mspace attributes, percentages are interpreted relative to the sizes of a block container. The spec instead says they are treated as 0: https://mathml-refresh.github.io/mathml-core/#space-mspace (this is consistent with MathML3 where the default size is 0x0)

@rwlbuis
Copy link
Contributor

rwlbuis commented Feb 24, 2020

cc @bfgeek @rwlbuis
This is related to CSS issues w3c/mathml#45 w3c/mathml#50 w3c/mathml#49 w3c/mathml#13

Rob has implemented the mspace element by mapping (in C++):
CSS vertical-align: height attribute value

This turned out to be problematic and was changed to be all internal and not visible for web developers:
https://chromium-review.googlesource.com/c/chromium/src/+/2068599
So now MathML does nothing special for vertical-align. Of course the rest of this issue still stands.

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

2 participants