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

Add subfeature for deprecated named values of the mathsize attribute #17921

Merged
merged 4 commits into from Sep 30, 2022

Conversation

fred-wang
Copy link
Contributor

@fred-wang fred-wang commented Sep 30, 2022

Summary

Add subfeature for deprecated named values "small", "normal", "big" of the mathsize attribute. These were removed in Firefox 83 [1] but are still supported by WebKit [2]. Also use "mirror" for Safari iOS, it has the same MathML support as macOS.

Test results and supporting details

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1664488
[2] https://github.com/WebKit/WebKit/blob/cd711e64b46522100059a32e3025f7875e4861c0/Source/WebCore/mathml/MathMLElement.cpp#L117

Related issues

N/A

Add subfeature for deprecated named values "small", "normal", "big"
of the mathsize attribute. These were removed in Firefox 83 [1] but
are still supported by WebKit [2].

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1664488
[2] https://github.com/WebKit/WebKit/blob/cd711e64b46522100059a32e3025f7875e4861c0/Source/WebCore/mathml/MathMLElement.cpp#L117
@github-actions github-actions bot added the data:mathml ➗ Compat data for MathML features. https://developer.mozilla.org/docs/Web/MathML label Sep 30, 2022
@fred-wang
Copy link
Contributor Author

cc @Elchi3

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When grouping features, please provide renderable description (otherwise small_normal_big will be rendered on data consumer's output, like the MDN compat tables).

mathml/global_attributes.json Show resolved Hide resolved
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about suggesting it at the wrong position. Thanks for making the update! 👍

@Elchi3 Elchi3 merged commit 36a519c into mdn:main Sep 30, 2022
@fred-wang fred-wang deleted the mathsize-named-values branch September 30, 2022 11:39
@fred-wang
Copy link
Contributor Author

@Elchi3 no worries. thanks for the review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:mathml ➗ Compat data for MathML features. https://developer.mozilla.org/docs/Web/MathML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants