Bug 246646
| Summary: | [JSC] Intl.NumberFormat ignores maximumFractionDigits with compact notation (both currency and decimal) | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Max Starets <mstarets> |
| Component: | JavaScriptCore | Assignee: | Yusuke Suzuki <ysuzuki> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | cunha.pedro.m, karlcow, mmaxfield, webkit-bug-importer, ysuzuki |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar |
| Version: | Safari 16 | ||
| Hardware: | Mac (Intel) | ||
| OS: | macOS 12 | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=147605 | ||
Max Starets
To reproduce, try the following:
new Intl.NumberFormat('en-US', {style: 'currency', currency: 'USD', notation: 'compact', maximumFractionDigits: 2, compactDisplay: 'long'}).format(97896)
Safari output: "$98K"
Firefox and Chrome output: "$97.90K"
new Intl.NumberFormat('en-US', {style: 'decimal', notation: 'compact', maximumFractionDigits: 2, compactDisplay: 'long'}).format(97896)
Safari output: "98 thousand"
Firefox and Chrome output: "97.9 thousand"
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Karl Dubost
There is also no tests on WPT covering this: Intl.NumberFormat
https://wpt.fyi/results/?label=master&label=experimental&aligned&view=subtest&q=maximumfractiondigits
Yusuke Suzuki
(In reply to Karl Dubost from comment #1)
> There is also no tests on WPT covering this: Intl.NumberFormat
> https://wpt.fyi/results/
> ?label=master&label=experimental&aligned&view=subtest&q=maximumfractiondigits
Intl.NumberFormat is not web API. It is test262's job to cover that.
Yusuke Suzuki
We should apply the spec side change.
Radar WebKit Bug Importer
<rdar://problem/101298045>
Yusuke Suzuki
Pull request: https://github.com/WebKit/WebKit/pull/5501
EWS
Committed 255691@main (9c113c3fb2ab): <https://commits.webkit.org/255691@main>
Reviewed commits have been landed. Closing PR #5501 and removing active labels.
Yusuke Suzuki
*** Bug 249231 has been marked as a duplicate of this bug. ***