1589095 - Add support for "type" and "style" options to Intl.ListFormat and enable Intl.ListFormat by default

Status: RESOLVED FIXED

(Core :: JavaScript: Internationalization API, enhancement, P3)

Description

[André Bargull [:anba]]

This is currently blocked because of missing ICU APIs: https://unicode-org.atlassian.net/browse/ICU-12863

Comment 1

[André Bargull [:anba]]

Attachment: Bug 1589095 - Part 1: Call new ICU API for list formatters to set style/width options. r=jwalden!

ulistfmt_openForType is a draft API, so it may not be available when compiling
against a system ICU. Instead of disabling Intl.ListFormat completely, this
patch adds a boolean parameter to InitializeListFormat to conditionally enable
the "type" and "style" options when ICU draft APIs are available.

Comment 2

[André Bargull [:anba]]

Attachment: Bug 1589095 - Part 2: Enable additional Intl.ListFormat test cases. r=jwalden!

"conjunction-long.js" can be removed, because the more thorough
"conjunction-type.js" test gets enabled in this patch.

Depends on D72916

Comment 3

[André Bargull [:anba]]

Attachment: Bug 1589095 - Part 3: Add ListFormat optimisation when list contains less two elements. r=jwalden!

We can avoid initialising the ICU list formatter and calling into C++ when the
list contains less than two elements.

Depends on D72917

Comment 4

[André Bargull [:anba]]

Attachment: Bug 1589095 - Part 4: Enable Intl.ListFormat by default. r=jwalden!

Depends on D72918

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a0fcee52ae04
Part 1: Call new ICU API for list formatters to set style/width options. r=jwalden
https://hg.mozilla.org/integration/autoland/rev/ed81e75b9623
Part 2: Enable additional Intl.ListFormat test cases. r=jwalden
https://hg.mozilla.org/integration/autoland/rev/26fe2fe0c9e5
Part 3: Add ListFormat optimisation when list contains less two elements. r=jwalden
https://hg.mozilla.org/integration/autoland/rev/f17442f0485c
Part 4: Enable Intl.ListFormat by default. r=jwalden

Comment 6

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/a0fcee52ae04
https://hg.mozilla.org/mozilla-central/rev/ed81e75b9623
https://hg.mozilla.org/mozilla-central/rev/26fe2fe0c9e5
https://hg.mozilla.org/mozilla-central/rev/f17442f0485c

Comment 7

[Florian Scholz (Open Web Docs)]

Announced on Firefox 78 for developers:
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/78#JavaScript

Reference docs:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat/ListFormat
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat/format
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat/formatToParts
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat/resolvedOptions
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat/supportedLocalesOf

Compat data update:
https://github.com/mdn/browser-compat-data/pull/6238