1548530 - Remove support for numalign/denomalign/align attributes

Status: RESOLVED FIXED

(Core :: MathML, task)

Description

[Frédéric Wang Nélar]

MathML CG decision: https://github.com/mathml-refresh/mathml/issues/30#issuecomment-487748458

Comment 1

[Frédéric Wang Nélar]

Attachment: data-collection-mathml-alignment-attributes.txt

Comment 2

[Frédéric Wang Nélar]

Attachment: Bug 1548530 - Remove support for numalign/denomalign/align attributes. r=emilio

Intent to unship: https://groups.google.com/forum/#!topic/mozilla.dev.platform/JnJVGTmIwPE

• Introduce a new preference option mathml.deprecated_alignment_attributes.disabled()
  to disable alignment attributes for mfrac/munder/mover/munderover elements.
• Disable the attributes in Nightly and when running WPT tests.
• Enable the attributes in other channels but add a counter and deprecation warning.
• Remove failure expectation for WPT test frac-numalign-denomalign-001.html for mfrac
• Add new WPT test for underover-legacy-align-attribute-001.html for munder/mover/munderover
• Enable the attributes for MathML reftests checking these attributes.
• Disable numalign/denomalign test for Mochitest test_bug975681.html when the attributes
  are disabled.

Comment 3

[Ben Miroglio [:bmiroglio]]

Comment on attachment 9094402 [details]
data-collection-mathml-alignment-attributes.txt

# Data Review Form 

1) Is there or will there be **documentation** that describes the schema for the ultimate data set in a public, complete, and accurate way? (see [here](https://github.com/mozilla/activity-stream/blob/master/docs/v2-system-addon/data_dictionary.md), [here](https://github.com/mozilla-mobile/focus/wiki/Install-and-event-tracking-with-the-Adjust-SDK), and [here](https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/index.html) for examples).  Refer to the appendix for "documentation" if more detail about documentation standards is needed.

This will be documented in the [probe dictionary](https://telemetry.mozilla.org/probe-dictionary/).

2) Is there a control mechanism that allows the user to turn the data collection on and off? (Note, for data collection not needed for security purposes, Mozilla provides such a control mechanism) Provide details as to the control mechanism available.

Yes, this can be turned off in Firefox Preferences.

3) If the request is for permanent data collection, is there someone who will monitor the data over time?

This is not permanent, although there's the potential for renewal. :fredw is responsible should a renewal be necessary.

4) Using the **[category system of data types](https://wiki.mozilla.org/Firefox/Data_Collection)** on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1: Technical Data

5) Is the data collection request for default-on or default-off?

default-on

6) Does the instrumentation include the addition of **any *new* identifiers** (whether anonymous or otherwise; e.g., username, random IDs, etc.  See the appendix for more details)?

No.

7) Is the data collection covered by the existing Firefox privacy notice? 

Yes.

8) Does there need to be a check-in in the future to determine whether to renew the data? 

Yes. In 6 months :fredw should check in.

9) Does the data collection use a third-party collection tool? **If yes, escalate to legal.**

No.

data-review: r+

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8fb0766d0ab2
Remove support for numalign/denomalign/align attributes. r=emilio

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/19328 for changes under testing/web-platform/tests

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 7

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/8fb0766d0ab2

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 9

[Kohei Yoshino [:kohei]]

Posted site compatibility note: https://www.fxsitecompat.dev/en-CA/docs/2019/various-legacy-mathml-features-have-been-deprecated/

Comment 10

[Florian Scholz (Open Web Docs)]

Updated:
https://developer.mozilla.org/en-US/docs/Web/MathML/Attribute
https://developer.mozilla.org/en-US/docs/Web/MathML/Element/munder
https://developer.mozilla.org/en-US/docs/Web/MathML/Element/mover
https://developer.mozilla.org/en-US/docs/Web/MathML/Element/munderover
https://wiki.developer.mozilla.org/en-US/docs/Web/MathML/Element/mfrac

Please add 'dev-doc-needed' to the bug that removes this in a stable Firefox version.