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

Update Intl.DateTimeFormat to reflect new behaviour of hourCycle option #33343

Merged
merged 3 commits into from
May 1, 2024

Conversation

ben-allen
Copy link
Contributor

Description

Updates the description of hourCycle option to reflect the current spec, which uses locale-dependent data to determine what hour cycles should be used for each locale when hourCycle is set.

Motivation

tc39/ecma402#758 changed how this option is interpreted; previous versions used very buggy algorithm to determine which hour cycle to use.

Note that no locale uses the "h24" hour cycle, and only one region (JP) uses "h11".

@ben-allen ben-allen requested a review from a team as a code owner May 1, 2024 00:10
@ben-allen ben-allen requested review from Josh-Cena and removed request for a team May 1, 2024 00:10
@github-actions github-actions bot added Content:JS JavaScript docs size/xs [PR only] 0-5 LoC changed labels May 1, 2024
Copy link
Contributor

github-actions bot commented May 1, 2024

Preview URLs

(comment last updated: 2024-05-01 00:35:41)

ben-allen and others added 2 commits April 30, 2024 17:13
…imeformat/datetimeformat/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Thanks!

@Josh-Cena Josh-Cena merged commit 7e059cd into mdn:main May 1, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:JS JavaScript docs size/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants