Skip to content

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

Merged
Josh-Cena merged 3 commits intomdn:mainfrom
ben-allen:intl-datetimeformat-hour12-changes
May 1, 2024
Merged

Update Intl.DateTimeFormat to reflect new behaviour of hourCycle option#33343
Josh-Cena merged 3 commits intomdn:mainfrom
ben-allen:intl-datetimeformat-hour12-changes

Conversation

@ben-allen
Copy link
Copy Markdown
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
@github-actions
Copy link
Copy Markdown
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
Copy Markdown
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
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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