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

Add support for -u extension 'rg' #950

Open
FrankYFTang opened this issue Jul 18, 2024 · 5 comments
Open

Add support for -u extension 'rg' #950

FrankYFTang opened this issue Jul 18, 2024 · 5 comments
Labels
c: meta Component: intl-wide issues s: comment Status: more info is needed to move forward User Preferences Related to user preferences

Comments

@FrankYFTang
Copy link
Contributor

This is an issue fork out from @anba's comment in the review of
tc39/proposal-intl-locale-info#83

During 2024-07-18, TG2 believe the patch @anba suggested has other undesireable side effect, such as "fr-u-ca-hebrew" may not return the right result .

What @anba suggest is

-        1. Let _r_ be a record whose fields are defined by <emu-xref href="#table-locale-weekinfo-record"></emu-xref>, with values based on _locale_.
+        1. Let _languageId_ be the longest prefix of _locale_ matched by the <code>unicode_language_id</code> Unicode locale nonterminal.
 +       1. Let _r_ be a record whose fields are defined by <emu-xref href="#table-locale-weekinfo-record"></emu-xref>, with values based on _languageId_.

but that remove not just 'rg' but also 'ca'

The question is how should we address the issue that we want to make sure right now we will not listen to 'rg' but will still respect to 'ca'.

@sffc

@FrankYFTang
Copy link
Contributor Author

Regarding should the calendar value in -u-ca- beside iso8601 impact the first day of week. I filed a CLDR ticket at https://unicode-org.atlassian.net/browse/CLDR-17819

@sffc
Copy link
Contributor

sffc commented Jul 19, 2024

According to UTS 35, https://unicode.org/reports/tr35/tr35-dates.html#first-day-overrides, which was added as part of https://unicode-org.atlassian.net/browse/CLDR-16866, -u-ca can impact the first day of week, in the following order:

  1. -u-fw
  2. -u-rg
  3. -u-ca
  4. region
  5. -u-sd

And there is also https://unicode-org.atlassian.net/browse/CLDR-17151 to promote the -u-sd to be higher up.

@FrankYFTang
Copy link
Contributor Author

FrankYFTang commented Jul 30, 2024

The first day of week will respect rg after ICU75 (see unicode-org/icu#2711 ) currently v8 is still on 74.2

@FrankYFTang
Copy link
Contributor Author

be aware of unicode-org/cldr#4049

@FrankYFTang FrankYFTang transferred this issue from tc39/proposal-intl-locale-info Dec 18, 2024
@FrankYFTang FrankYFTang changed the title Ensure getWeekInfo will not not listen to 'rg' Add support for -u extension 'rg' Dec 18, 2024
@FrankYFTang
Copy link
Contributor Author

This issue is transfered from Intl Locale Info API to ECMA402 because the nature of the issue deal with the "rg" extension and require an overall discussion beyond the Intl Locale API itself.

@sffc sffc added c: meta Component: intl-wide issues s: comment Status: more info is needed to move forward User Preferences Related to user preferences labels Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: meta Component: intl-wide issues s: comment Status: more info is needed to move forward User Preferences Related to user preferences
Projects
None yet
Development

No branches or pull requests

2 participants