Skip to content

Normative: Intl Locale API proposal for Stage 4#942

Merged
gibson042 merged 40 commits intotc39:mainfrom
FrankYFTang:LocaleInfoStage4
Feb 19, 2026
Merged

Normative: Intl Locale API proposal for Stage 4#942
gibson042 merged 40 commits intotc39:mainfrom
FrankYFTang:LocaleInfoStage4

Conversation

@FrankYFTang
Copy link
Copy Markdown
Contributor

The spec text change for https://github.com/tc39/proposal-intl-locale-info

We should discuss this once Intl Locale Info API is ready to move to stage 4.
This is one of the require task.

@sffc
Copy link
Copy Markdown
Contributor

sffc commented Dec 21, 2024

@ryzokuken ryzokuken marked this pull request as draft May 13, 2025 14:21
@FrankYFTang FrankYFTang moved this to Priority Issues in ECMA-402 Meeting Topics Oct 8, 2025
@sffc sffc moved this from Priority Issues to Previously Discussed in ECMA-402 Meeting Topics Oct 9, 2025
@sffc
Copy link
Copy Markdown
Contributor

sffc commented Oct 13, 2025

@FrankYFTang FrankYFTang marked this pull request as ready for review October 14, 2025 23:27
@FrankYFTang
Copy link
Copy Markdown
Contributor Author

Ready to be review for merging if Intl Locale Info advanced to Stage 4. Still need to merge tc39/proposal-intl-locale-info#92 into this PR first if TC39 approve both

@FrankYFTang
Copy link
Copy Markdown
Contributor Author

@ben-allen @gibson042 @ryzokuken will be nice if you can voice your concern or support about this PR from the editor's view ASAP before the TG1. Thanks

Copy link
Copy Markdown
Member

@gibson042 gibson042 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few things to fix before approval.

spec/locale.html Outdated
Comment on lines +396 to +397
1. Let _region_ be GetLocaleRegion(_loc_.[[Locale]]).
1. If _region_ is *undefined*, return *undefined*.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This logic seems like it belongs in TimeZonesOfLocale, replacing the assertion and bringing it in better alignment with LocaleDirection.

         1. Let _region_ be GetLocaleRegion(_loc_.[[Locale]]).
-        1. Assert: _region_ is not *undefined*.
+        1. If _region_ is *undefined*, return *undefined*.
         1. Let _list_ be a List of unique canonical time zone identifiers, which must be String values indicating a canonical Zone name of the IANA Time Zone Database, of those in common use in _region_. The list is empty if no time zones are commonly used in _region_. The list is sorted according to lexicographic code unit order.
         1. Return CreateArrayFromList( _list_ ).
Suggested change
1. Let _region_ be GetLocaleRegion(_loc_.[[Locale]]).
1. If _region_ is *undefined*, return *undefined*.

FrankYFTang and others added 12 commits November 17, 2025 19:21
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
@FrankYFTang
Copy link
Copy Markdown
Contributor Author

PTAL
I adress your feedback
I also merge the last PR in the proposal

and in 2025-11-18 TC39 this stage 4 proposal is advanced to stage 4

FrankYFTang and others added 3 commits November 19, 2025 16:20
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
@FrankYFTang
Copy link
Copy Markdown
Contributor Author

PTAL

FrankYFTang and others added 10 commits November 20, 2025 14:53
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
@sffc
Copy link
Copy Markdown
Contributor

sffc commented Feb 19, 2026

Copy link
Copy Markdown
Member

@gibson042 gibson042 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few editorial nits that I'd be happy to address myself, but this PR cannot merge until we resolve the getTextInfo output question.

FrankYFTang and others added 8 commits February 19, 2026 13:22
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
@FrankYFTang
Copy link
Copy Markdown
Contributor Author

PTAL

@gibson042 gibson042 merged commit 70b0ecc into tc39:main Feb 19, 2026
2 checks passed
@FrankYFTang FrankYFTang deleted the LocaleInfoStage4 branch February 20, 2026 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Previously Discussed

Development

Successfully merging this pull request may close these issues.

3 participants