Add Intl.Locale.getWeekInfo() returns minimalDays data
#26793
Conversation
github-actions
bot
added
data:js
|
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs). |
| }, | ||
| "getWeekInfo": { | ||
| "__compat": { | ||
| "description": "`getWeekInfo()` method", |
There was a problem hiding this comment.
This looks weird. Virtually no method uses this kind of description—is that going to change en masse?
There was a problem hiding this comment.
Doesn't look weird to me, and isn't uncommon: https://github.com/search?q=repo%3Amdn%2Fbrowser-compat-data%20%2F%20method%22%2F&type=code
What is your concern with this description?
There was a problem hiding this comment.
Nothing, it's just inconsistent with the majority which don't have descriptions. If methods are supposed to have descriptions like this (which I'm okay with) we should do that consistently.
There was a problem hiding this comment.
Just a quick follow-up: We don't have plans to change this en masse. We probably want to add feature types at some point, so we can generate these at build time. Until then, adding descriptions manually makes the feature look slightly nicer in the BCD tables, with the risk of causing a line wrap.
3 participants
Summary
Adds behavioral subfeature for
Intl.Locale.getWeekInfo()returningminimalDays.Test results and supporting details
Related issues
Fixes #26766.