Skip to content
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

"standard" and "search" collations in CollationsOfLocale() #31

Closed
anba opened this issue May 14, 2021 · 0 comments
Closed

"standard" and "search" collations in CollationsOfLocale() #31

anba opened this issue May 14, 2021 · 0 comments

Comments

@anba
Copy link
Contributor

anba commented May 14, 2021

Per https://tc39.es/proposal-intl-locale-info/#sec-hour-cycles-of-locale:

Let list be a List of 1 or more collation identifiers, which must be String values conforming to the type sequence from UTS 35 Unicode Locale Identifier, section 3.2, sorted in descending preference of those in common use in the locale for string comparison.

Intl.Collator suppresses "standard" and "search" collations, because they're handled through the "usage" property. I'm not sure the same reasoning also applies to CollationsOfLocale(), especially because the current spec text describes that the commonly used collations should be returned, which normally do include "standard" and "search".

However we decide to handle those two collation identifiers, we should definitely explicitly mention them in CollationsOfLocale().

FrankYFTang added a commit that referenced this issue May 20, 2021
Address #31
This PR is logical because in https://tc39.es/ecma402/#sec-intl-collator-internal-slots
```
The values "standard" and "search" must not be used as elements in any [[SortLocaleData]].[[<locale>]].[[co]] and [[SearchLocaleData]].[[<locale>]].[[co]] list.
```
FrankYFTang added a commit that referenced this issue May 25, 2021
Address #31
This PR is logical because in https://tc39.es/ecma402/#sec-intl-collator-internal-slots
```
The values "standard" and "search" must not be used as elements in any [[SortLocaleData]].[[<locale>]].[[co]] and [[SearchLocaleData]].[[<locale>]].[[co]] list.
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants