Skip to content

Commit e2f11a4

Browse files
fred-wangmoz-wptsync-bot
authored andcommitted
Bug 1636693 [wpt PR 23495] - Update menclose help and explains better how tests work in MathML core, a=testonly
Automatic update from web-platform-tests Update menclose help and explain better how tests work in MathML core (#23495) No consensus have been reached on the future on menclose so far but this element has been the source of confusion so the current draft text has been removed from MathML Core and put in a separate pull request: w3c/mathml-core#24 This PR updates the WPT tests accordingly: * Actual tests for menclose are renamed "tentative" and refer to the pull request above, so that it becomes clear they are not definitive yet and browsers are free to ignore them. * Tests for mrow-like are now referring to MathML3 https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose They are still valid because unknown elements are treated as mrow-like in MathML Core. * Tests for removed legacy features are obviously still valid too. Again, the documentation is updated to refer to MathML3 https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose -- wpt-commits: 46b884903ba0153b2cd9d932504f6b88f57614cf wpt-pr: 23495
1 parent 1aa0997 commit e2f11a4

9 files changed

+13
-9
lines changed

testing/web-platform/tests/mathml/presentation-markup/menclose/legacy-menclose-radical-notation.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<meta charset="utf-8"/>
55
<title>Legacy menclose radical notation</title>
66
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
7-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
7+
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
88
<meta name="assert" content="Verify that the legacy menclose radical notation is ignored.">
99
<link rel="match" href="legacy-menclose-radical-notation-ref.html">
1010
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />

testing/web-platform/tests/mathml/presentation-markup/mrow/inferred-mrow-baseline.html

+2-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#error-message-merror">
1010
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#making-sub-expressions-invisible-mphantom">
1111
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#the-top-level-math-element">
12-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
12+
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
1313
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#adjust-space-around-content-mpadded">
1414
<meta name="assert" content="Baseline for mrow-like elements is correct.">
1515
<script src="/mathml/support/feature-detection.js"></script>
@@ -41,6 +41,7 @@
4141
<math><merror><mspace id="aboveError" width="10px" height="30px" style="background: purple"></mspace><mspace id="belowError" width="10px" depth="30px" style="background: blue"></mspace></merror></math>
4242
<math><mphantom><mspace style="visibility: visible;" id="abovePhantom" width="10px" height="30px" style="background: purple"></mspace><mspace style="visibility: visible;" id="belowPhantom" width="10px" depth="30px" style="background: blue"></mspace></mphantom></math>
4343
<math><mspace id="aboveMath" width="10px" height="30px" style="background: purple"></mspace><mspace id="belowMath" width="10px" depth="30px" style="background: blue"></mspace></math>
44+
<!-- menclose is treated as <unknown> in MathML Core -->
4445
<math><menclose notation="box"><mspace id="aboveMenclose" width="10px" height="30px" style="background: purple"
4546
></mspace><mspace id="belowMenclose" width="10px" depth="30px" style="background: blue"></mspace></menclose></math>
4647
<math><mpadded lspace="10px"><mspace id="aboveMpadded" width="10px" height="30px" style="background: purple"

testing/web-platform/tests/mathml/presentation-markup/mrow/inferred-mrow-stretchy.html

+2-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#error-message-merror">
1010
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#making-sub-expressions-invisible-mphantom">
1111
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#the-top-level-math-element">
12-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
12+
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
1313
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#adjust-space-around-content-mpadded">
1414
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#operator-fence-separator-or-accent-mo">
1515
<meta name="assert" content="Operators can stretch inside mrow-like elements.">
@@ -51,6 +51,7 @@
5151
<math><merror><mo id="moError">&#x21A8;</mo><mspace width="1px" height="100px" style="background: magenta"></mspace></merror></math>
5252
<math><mphantom><mo style="visibilty: visible;" id="moPhantom">&#x21A8;</mo><mspace width="1px" height="100px" style="background: magenta"></mspace></mphantom></math>
5353
<math><mo id="moMath">&#x21A8;</mo><mspace width="1px" height="100px" style="background: magenta"></mspace></math>
54+
<!-- menclose is treated as <unknown> in MathML Core -->
5455
<math><menclose notation="box"><mo id="moMenclose">&#x21A8;</mo><mspace width="1px" height="100px" style="background: magenta"></mspace></menclose></math>
5556
<math><mpadded lspace="10px"><mo id="moMpadded">&#x21A8;</mo><mspace width="1px" height="100px" style="background: magenta"></mspace></mpadded></math>
5657
<math><unknown><mo id="moUnknown">&#x21A8;</mo><mspace width="1px" height="100px" style="background: magenta"></mspace></unknown></math>

testing/web-platform/tests/mathml/presentation-markup/mrow/legacy-mstyle-attributes.html

+2-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#fractions-mfrac">
1010
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#operator-fence-separator-or-accent-mo">
1111
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#string-literal-ms">
12-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
12+
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
1313
<meta name="assert" content="Legacy mstyle attributes are ignored">
1414
<script src="/resources/testharness.js"></script>
1515
<script src="/resources/testharnessreport.js"></script>
@@ -135,6 +135,7 @@
135135
</mstyle>
136136
</math>
137137
</p>
138+
<!-- notation attribute is from MathML3's menclose element -->
138139
<p class="TestContainer" id="menclose">
139140
<math>
140141
<mstyle notation="box">

testing/web-platform/tests/mathml/presentation-markup/mrow/spacing.html

+2-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#error-message-merror">
1010
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#making-sub-expressions-invisible-mphantom">
1111
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#the-top-level-math-element">
12-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
12+
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
1313
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#adjust-space-around-content-mpadded">
1414
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#operator-fence-separator-or-accent-mo">
1515
<meta name="assert" content="Spacing is added around operators inside mrow-like elements.">
@@ -43,6 +43,7 @@
4343
<math><merror id="Error"><mn>1</mn><mo lspace="50px"></mo><mn>2</mn></merror></math>
4444
<math><mphantom id="Phantom"><mn>1</mn><mo lspace="50px">|</mo><mn>2</mn></mphantom></math>
4545
<math id="Math"><mn>1</mn><mo lspace="50px">|</mo><mn>2</mn></math>
46+
<!-- menclose is treated as <unknown> in MathML Core -->
4647
<math><menclose id="Menclose" notation="box"><mn>1</mn><mo lspace="50px">|</mo><mn>2</mn></menclose></math>
4748
<math><mpadded id="Mpadded" lspace="10px"><mn>1</mn><mo lspace="50px">|</mo><mn>2</mn></mpadded></math>
4849
<math><unknown id="Unknown"><mn>1</mn><mo lspace="50px">|</mo><mn>2</mn></unknown></math>

testing/web-platform/tests/mathml/relations/css-styling/color-004.html testing/web-platform/tests/mathml/relations/css-styling/color-004.tentative.html

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
<title>color</title>
66
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#css-styling">
77
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#text-mtext">
8-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
9-
<link rel="match" href="color-004-ref.html"/>
8+
<link rel="help" href="https://github.com/mathml-refresh/mathml-core/pull/24">
9+
<link rel="match" href="color-004.tentative-ref.html"/>
1010
<meta name="assert" content="Verify that the color is used for text and graphical elements of the menclose element.">
1111
</head>
1212
<body>

testing/web-platform/tests/mathml/relations/css-styling/visibility-004.html testing/web-platform/tests/mathml/relations/css-styling/visibility-004.tentative.html

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
<title>visibility</title>
66
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#css-styling">
77
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#text-mtext">
8-
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
9-
<link rel="match" href="visibility-004-ref.html"/>
8+
<link rel="help" href="https://github.com/mathml-refresh/mathml-core/pull/24">
9+
<link rel="match" href="visibility-004.tentative-ref.html"/>
1010
<meta name="assert" content="Verify that visibility=hidden is used for the text and graphical elements of the menclose element.">
1111
</head>
1212
<body>

0 commit comments

Comments
 (0)