-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Intl.Locale info getters changed to methods #26288
Comments
We have to keep the property pages as it is till two years after all the browsers drop support. [ref] Instead we need to create new pages for methods. And put warnings on property pages saying use methods now. |
Yes, that's the plan I think. Can't remove them :/ |
An alternative is to keep one page and say they are methods, but some browsers implement them as properties (and once all have updated their implementation, only keep it into bcd. |
Not sure what I should do: redirect, or new pages? I am leaning to redirects because more pages is more trouble. Just want to make sure that everyone's comfortable with this—if so, the BCD will be changed by moving the feature and adding a note saying in certain versions it's implemented under the alternative name as an accessor property. |
I'm comfortable with this and I think we have acted similar on previous cases like this. |
This should be fixed now |
MDN URL
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Locale
What information was incorrect, unhelpful, or incomplete?
The mention of the following getters:
calendars
,collations
,hourCycles
,numberingSystems
,timeZones
,textInfo
,weekInfo
.What did you expect to see?
They have been changed into methods: tc39/proposal-intl-locale-info#67
The spec URLs have been removed in BCD: mdn/browser-compat-data#19392
Do you have any supporting links, references, or citations?
This change has been implemented in JSC: WebKit/WebKit#10241
V8 bug: https://bugs.chromium.org/p/v8/issues/detail?id=13835&q=locale%20info&can=2
SpiderMonkey doesn't implement the locale-info proposal yet.
Do you have anything more you want to share?
No response
The text was updated successfully, but these errors were encountered: