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
Re-introduce subfeatures for MathML attributes superscriptshift/subsc… #17962
Conversation
…riptshift Re-introduce subfeatures for MathML attributes superscriptshift/subscriptshift. These were incorrectly removed in [1]. Actually, they were disabled in Firefox in [2] and implemented in WebKit in [3]. Also make mirror Safari iOS mirror Safari (they have the same MathML support). [1] mdn#17617 [2] https://bugzilla.mozilla.org/show_bug.cgi?id=1664488 [3] https://commits.webkit.org/139551@main
cc @Elchi3 |
mathml/elements/msubsup.json
Outdated
"opera": "mirror", | ||
"opera_android": "mirror", | ||
"safari": { | ||
"version_added": false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Safari is false here and for superscriptshift
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's a mistake
…hift. Document deprecated MathML attributes superscriptshift/subscriptshift. These were incorrectly removed in [3]. Actually, they were disabled in Firefox in [1] and implemented in WebKit in [2]. See also [4]. [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1664488 [2] https://commits.webkit.org/139551@main [3] mdn@21f8488 [4] mdn/browser-compat-data#17962
mathml/elements/mmultiscripts.json
Outdated
"opera": "mirror", | ||
"opera_android": "mirror", | ||
"safari": { | ||
"version_added": "9" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this 9 and all other safari versions are 7?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be 7 too. Sorry...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 👍
…hift. (#21452) Document deprecated MathML attributes superscriptshift/subscriptshift. These were incorrectly removed in [3]. Actually, they were disabled in Firefox in [1] and implemented in WebKit in [2]. See also [4]. [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1664488 [2] https://commits.webkit.org/139551@main [3] 21f8488 [4] mdn/browser-compat-data#17962
Summary
Re-introduce subfeatures for MathML attributes superscriptshift/subscriptshift. These were incorrectly removed in [3]. Actually, they were disabled in Firefox in [1] and implemented in WebKit in [2]. Also make mirror Safari iOS mirror Safari (they have the same MathML support).
Test results and supporting details
[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1664488
[2] https://commits.webkit.org/139551@main
Related issues
[3] #17617