-
Notifications
You must be signed in to change notification settings - Fork 2k
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 real values for all MathML features #3590
Conversation
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.
If it’s not supported on Desktop, then it’s definitely not supported on Mobile.
67a6814
to
0ee9b9c
Compare
Thanks @ExE-Boss, I've updated the PR. |
Maybe @fred-wang can help reviewing or knows someone who could. |
IIUC Microsoft Edge uses different web engines on mobile (e.g. blink on android and webkit on iOS). Also Microsoft plans to switch to blink on Windows too, so that would mean the same result as Chrome ( https://blogs.windows.com/windowsexperience/2018/12/06/microsoft-edge-making-the-web-better-through-more-open-source-collaboration/ ). I don't know the exact versions for firefox/safari. I guess it is fine to put these values and fix them later if people report errors... |
Except that |
Thanks @ExE-Boss and @fred-wang, let's merge this then. |
The first two commits indicate no MathML support at all in Edge or Edge Mobile
See https://developer.microsoft.com/en-us/microsoft-edge/platform/status/mathml/
The third commit also updates the few Firefox and Safari values with real values.
With this PR, 100% of the data in the mathml/ folder has real values for the 8 tracked browsers.
Overall, it changes the stats a bit as well:
Before (current master):
true
valuesnull
valuesAfter this PR:
true
valuesnull
values