-
Notifications
You must be signed in to change notification settings - Fork 22.4k
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 Intl.DateTimeFormat.resolvedOptions dateStyle, timeStyle #33634
Conversation
...s/en-us/web/javascript/reference/global_objects/intl/datetimeformat/resolvedoptions/index.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! I'm in the process of updating these docs but it's good to make them more correct. I just have one question.
...s/en-us/web/javascript/reference/global_objects/intl/datetimeformat/resolvedoptions/index.md
Outdated
Show resolved
Hide resolved
Preview URLs
Flaws (1)Note! 1 document with no flaws that don't need to be listed. 🎉 URL:
External URLs (1)URL:
(comment last updated: 2024-06-10 18:56:07) |
...s/en-us/web/javascript/reference/global_objects/intl/datetimeformat/resolvedoptions/index.md
Outdated
Show resolved
Hide resolved
- Clarify dateStyle and timeStyle - Add example - Add missing hourCycle - Clarify calendar and numberingSystem
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks! As I mentioned, I already have stashed work locally that aims to unify the format of all resolvedOptions
pages, so I took this opportunity to apply that format. I have made sure to incorporate your changes.
Also as suggested I updated the constructor page slightly. I hope everything looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better wording, thanks! Flagged a couple of potential issues inline.
files/en-us/web/javascript/reference/global_objects/intl/datetimeformat/datetimeformat/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/javascript/reference/global_objects/intl/datetimeformat/datetimeformat/index.md
Show resolved
Hide resolved
...s/en-us/web/javascript/reference/global_objects/intl/datetimeformat/resolvedoptions/index.md
Outdated
Show resolved
Hide resolved
Is this ready for merging? |
I think so. Thanks! |
Great, welcome aboard! 😄 |
Description
Document
dateStyle
andtimeStyle
forIntl.DateTimeFormat.resolvedOptions()
return value.Motivation
These properties were added in tc39/ecma402#457, but are not covered by the current documentation.
Additional details
See current https://tc39.es/ecma402/#table-datetimeformat-resolvedoptions-properties
Related issues and pull requests