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

Change definition of space-like maction and semantics #160

Closed
fred-wang opened this issue Oct 16, 2019 · 4 comments
Closed

Change definition of space-like maction and semantics #160

fred-wang opened this issue Oct 16, 2019 · 4 comments
Labels
MathML Core Issues affecting the MathML Core specification

Comments

@fred-wang
Copy link

Defintion https://mathml-refresh.github.io/mathml-core/#dfn-space-like has the following:

  • Or an unknown MathML element, <mrow>, <mstyle>, <mphantom> or <mpadded> all of whose in-flow children are space-like;
  • An <maction> or <semantics> element whose first in-flow child exists and is space-like.

Test: mathml/presentation-markup/spaces/space-like-002.html

When trying to make maction/semantics behave as mrow in https://bugzilla.mozilla.org/show_bug.cgi?id=1588733 ; this test now fails.

Proposal: merge these two items:

  • Or an unknown MathML element, <mrow>, <maction>, <semantics>, <mstyle>, <mphantom> or <mpadded> all of whose in-flow children are space-like;

Since normally non-visible maction/semantics children are displayed "none", they are effectively out-of-flow and the MathML3 logic is preserved.

@fred-wang fred-wang added MathML Core Issues affecting the MathML Core specification need tests Issues related to writing WPT tests need resolution Issues needing resolution at MathML Refresh CG meeting need specification update Issues requiring specification changes labels Oct 16, 2019
@davidcarlisle
Copy link
Collaborator

This seems reasonable to me

@fred-wang
Copy link
Author

Consensus from 2019/11/11: Handle maction/semantics the same as other mrow-like ("whose in-flow children are space-like" instead of "whose first in-flow child exists and is space-like")

@NSoiffer
Copy link
Contributor

@fred-wang: can this be closed due to #182 and #183?

@fred-wang
Copy link
Author

Tests updated for the new definition web-platform-tests/wpt#21568 (we could add more tests for semantics/maction with more than one child, but I guess that's enough for now).

@fred-wang fred-wang removed need specification update Issues requiring specification changes need tests Issues related to writing WPT tests labels Mar 22, 2020
@fred-wang fred-wang removed the need resolution Issues needing resolution at MathML Refresh CG meeting label Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MathML Core Issues affecting the MathML Core specification
Projects
None yet
Development

No branches or pull requests

3 participants