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

mpadded: prohibit vertical pseudo-units in horizontal attributes #81

Closed
fred-wang opened this issue Mar 23, 2019 · 8 comments
Closed

mpadded: prohibit vertical pseudo-units in horizontal attributes #81

fred-wang opened this issue Mar 23, 2019 · 8 comments
Labels
compatibility Issues affecting backward compatibility css / html5 Issues related to CSS or HTML5 interoperability MathML Core Issues affecting the MathML Core specification

Comments

@fred-wang
Copy link

cc @rwlbuis, @emilio

mpadded support pseudo-units https://mathml-refresh.github.io/mathml/chapter3.html#presm.mpaddedatt which allows width to depend on vertical metrics like this:

<mpadded width="200%height">

This is causing issue for calculating intrinsic width. I propose to add a restriction and to disallow vertical pseudo-units (height, voffset, depth) in horizontal attributes (lspace, width). Probably worth adding in #55 too

@fred-wang fred-wang added MathML Core Issues affecting the MathML Core specification css / html5 Issues related to CSS or HTML5 interoperability labels Mar 23, 2019
@fred-wang fred-wang added the compatibility Issues affecting backward compatibility label Mar 25, 2019
@fred-wang fred-wang added the need resolution Issues needing resolution at MathML Refresh CG meeting label May 16, 2019
@rwlbuis
Copy link

rwlbuis commented Jun 3, 2019

AP: contact Brian Kardell about this. Since I don't think he can be referenced here directly, I'll ping him internally through the Igalia infrastructure.

@rwlbuis
Copy link

rwlbuis commented Jun 3, 2019

Brian, I briefly looked into the above case where width depends on height. I thought of CSS calc, it can use vw/vh, but that is not quite the same. Can you think of some way to do this that would make the mpadded pseudo units unneeded?

@rwlbuis
Copy link

rwlbuis commented Jun 4, 2019

After discussing with Brian, he mentioned the upcoming aspect-ratio CSS property should fix the "set width using height" and "set height using width" problems:
https://www.smashingmagazine.com/2019/03/aspect-ratio-unit-css/
https://drafts.csswg.org/css-sizing-4/#ratios

@rwlbuis
Copy link

rwlbuis commented Jun 4, 2019

Note that aspect-ratio would not know about height/depth attributes/concepts as specified for mpadded, it would just know about content width and height.

@fred-wang
Copy link
Author

cc @distler

These are the possible non-length values that can be generated by itex2MML or TeXZilla:
lspace="-100%width" (\mathllap)
lspace="-50%width" (\mathclap)
height="+[LENGTH][UNIT]" (\mathraisebox)
depth="+[LENGTH][UNIT]" (\mathraisebox)
depth="depth" (\mathraisebox)

@fred-wang fred-wang removed the need resolution Issues needing resolution at MathML Refresh CG meeting label Jun 14, 2019
@fred-wang
Copy link
Author

We agreed to prohibit vertical pseudo-units in horizontal attributes and horizontal pseudo-units in vertical attributes during meeting from 10/06/2019.

@fred-wang
Copy link
Author

The core spec no longer refers to MathML 3 for the mpadded syntax and pseudo-units have been restricted:
https://mathml-refresh.github.io/mathml-core/#adjust-space-around-content-mpadded

@fred-wang fred-wang removed the need specification update Issues requiring specification changes label Jun 14, 2019
fred-wang added a commit to web-platform-tests/wpt that referenced this issue May 27, 2020
fred-wang added a commit to web-platform-tests/wpt that referenced this issue May 27, 2020
@fred-wang fred-wang removed the need tests Issues related to writing WPT tests label May 27, 2020
@fred-wang
Copy link
Author

tests added

moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Jun 1, 2020
…ng on orthogonal metrics are …, a=testonly

Automatic update from web-platform-tests
mpadded: verify allow pseudo-units relying on orthogonal metrics are forbidden (#23796)

See w3c/mathml#81
--

wpt-commits: ccdf5417be20f26de1ab65e7bd19384bf0efceab
wpt-pr: 23796
xeonchen pushed a commit to xeonchen/gecko that referenced this issue Jun 2, 2020
…ng on orthogonal metrics are …, a=testonly

Automatic update from web-platform-tests
mpadded: verify allow pseudo-units relying on orthogonal metrics are forbidden (#23796)

See w3c/mathml#81
--

wpt-commits: ccdf5417be20f26de1ab65e7bd19384bf0efceab
wpt-pr: 23796
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility Issues affecting backward compatibility css / html5 Issues related to CSS or HTML5 interoperability MathML Core Issues affecting the MathML Core specification
Projects
None yet
Development

No branches or pull requests

2 participants