Skip to content

Commit

Permalink
Remove support for the menclose's "radical" notation.
Browse files Browse the repository at this point in the history
See https://groups.google.com/forum/#!topic/mozilla.dev.platform/vwAkuZIEhnY

* Introduce a new preference option to disable menclose's "radical" notation.
* Disable the notation in Nightly and when running WPT tests.
* Enable the notation in other channels together with a counter and
  deprecation warning.
* Update WPT test legacy-menclose-radical-notation.html
  - Fix test: "radical" should be equivalent to "", which is not the same as
    the default value "longdiv".
    See w3c/mathml#144
  - Add a test "box radical" which should be equivalent to "box".
  - Remove failure expectation.
* Enable the radical notation for MathML reftests testing it.

Differential Revision: https://phabricator.services.mozilla.com/D46721

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1548522
gecko-commit: d2613eaa13690e3771e84ef6ff5080a8fa348a66
gecko-integration-branch: autoland
gecko-reviewers: emilio
  • Loading branch information
fred-wang authored and moz-wptsync-bot committed Oct 3, 2019
1 parent 29dca2c commit 7fc3108
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@
<menclose>
<mn>123</mn>
</menclose>
<menclose notation="box">
<mn>123</mn>
</menclose>
</math>
</body>
</html>
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<title>Legacy menclose radical notation</title>
<link rel="help" href="https://www.w3.org/TR/MathML3/chapter3.html#presm.menclose">
<link rel="help" href="https://mathml-refresh.github.io/mathml-core/#enclose-expression-inside-notation-menclose">
<meta name="assert" content="Verify that the legacy menclose radical notation is not supported.">
<meta name="assert" content="Verify that the legacy menclose radical notation is ignored.">
<link rel="match" href="legacy-menclose-radical-notation-ref.html">
<link rel="stylesheet" type="text/css" href="/fonts/ahem.css" />
<style>
Expand All @@ -19,6 +19,9 @@
<menclose notation="radical">
<mn>123</mn>
</menclose>
<menclose notation="box radical">
<mn>123</mn>
</menclose>
</math>

<script src="/mathml/support/feature-detection.js"></script>
Expand Down

0 comments on commit 7fc3108

Please sign in to comment.