Issue with "Intl.NumberFormat() constructor": (narrowSymbol not supported on Safari)

[cgarrovillo]

MDN URL: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat/NumberFormat

What information was incorrect, unhelpful, or incomplete?

The incompatibility table does not warn that narrowSymbol is not a valid option for Intl.NumberFormat

Specific section or headline?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat/NumberFormat#browser_compatibility

What did you expect to see?

A warning for narrowSymbol incompatibility

Did you test this? If so, how?

Visited the website

MDN Content page report details

• Folder: en-us/web/javascript/reference/global_objects/intl/numberformat/numberformat
• MDN URL: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat/NumberFormat
• GitHub URL: https://github.com/mdn/content/blob/main/files/en-us/web/javascript/reference/global_objects/intl/numberformat/numberformat/index.html
• Last commit: mdn/content@3b80812
• Document last modified: 2021-02-19T19:57:02.000Z

[sideshowbarker]

Do you have a test case that demonstrates the problem?

narrowSymbol can’t be used directly as a key/parameter-name in the options object you give to the Intl.NumberFormat() constructor, right? Instead narrowSymbol is a possible value for the currencyDisplay option — like this:

new Intl.NumberFormat('en-US', { style: 'currency', currency: 'USD', currencyDisplay: 'narrowSymbol' }).format(3500)

The relevant Test262 test at https://test262.report/browse/intl402/NumberFormat/currencyDisplay-unit.js shows OK for JavaScriptCore for that test.

[cgarrovillo]

I can drum up something end of day today. Not sure why test262 is reporting it that way.

[Ryuno-Ki]

Preliminary assigned this issue to you, then, @cgarrovillo

[cgarrovillo]

https://github.com/cgarrovillo/MDN-Intl.NumberFormat/tree/main

I just realized I didnt format the unit_amount in the screenshot. But you can see the error regardless.

[sideshowbarker]

OK, I’ve transferred this to the BCD repo, since that’s where we need to make a change.

@cgarrovillo Thanks yeah, using he https://github.com/cgarrovillo/MDN-Intl.NumberFormat/tree/main document, I’ve also reproduced this and confirmed in my environment that Safari 14 reports an error.

Interestingly, though, Safari Technology Preview (Release 119 (Safari 14.1, WebKit 16611.1.10.1)) doesn’t report an error. https://trac.webkit.org/changeset/266031/webkit shows that narrowSymbol support was added 2020-08-22. And Safari 14 was released less than a month later, 2020-09-16 — so the Safari 14 release branch was cut before this change went into the trunk.

So we should document this in BCD by marking the Safari support with partial_implementation and a note.

@cgarrovillo Do you want to take a shot at writing up patch/PR for that? (If not, I can do it.)

I guess one of us should also raise an issue at https://github.com/tc39/test262/issues

[cgarrovillo]

I could raise this issue over at test262.

[sideshowbarker]

OK, I’ve opened #9371 with an update for this.