-
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
Mark Intl.NumberFormat support partial in Safari #9371
Mark Intl.NumberFormat support partial in Safari #9371
Conversation
a213e1f
to
dc863bb
Compare
hm, so this talks about features that are also sub features in the data. I think this makes the table quite inconsistent. Like, we don't usually mark the main feature as partial_support when there are additional features below that indicate the support for what you talk about in partial support. Do I make sense? |
dc863bb
to
d3d7456
Compare
Code examination of https://trac.webkit.org/changeset/266031/webkit indicates that support for currencyDisplay: 'narrowSymbol' wasn’t added to WebKit until 2020-08-22 — and didn’t make it into Safari 14 (confirmed by testing), which was released 2020-09-16. Fixes mdn#9319
d3d7456
to
1a39a51
Compare
Yup — make sense; I’ve amended the patch accordingly |
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.
Description would be good for consistency with the other sub features. LGTM otherwise!
Co-authored-by: Florian Scholz <fscholz@mozilla.com>
Code examination of https://trac.webkit.org/changeset/266031/webkit indicates that support for
currencyDisplay: 'narrowSymbol'
wasn’t added to WebKit until 2020-08-22 — and didn’t make it into Safari 14 (confirmed by testing), released 2020-09-16.