Comments (15)
this is a difficult issue. What should we do in this case?
Should we do in constructor throw? (we will not throw for other key type
or should we not throw in the constructor but then what should we return if user call the firstDayOfWeek ?
from proposal-intl-locale-info.
@sffc @zbraniecki @littledan @anba need advise
from proposal-intl-locale-info.
Hmm, another edge case. This is a consequence of our departure from how other subtags work based on the feedback from TG1 that we should return integers rather than strings.
Current behavior of invalid values in other subtags:
// OK: gregory1 is not a real calendar but it is BCP-47
new Intl.Locale("en", { calendar: "gregory1" }).toString()
'en-u-ca-gregory1'
// Wrong: gregory12 is too long for BCP-47
new Intl.Locale("en", { calendar: "gregory12" }).toString()
// Uncaught RangeError: Incorrect locale information provided
The following cases should always be RangeError since they are BCP-47:
new Intl.Locale("en", { firstDayOfWeek: "mon1234567" })
(invalid BCP-47)new Intl.Locale("en-u-fw-mon1234567")
So, the cases we need to work out are:
new Intl.Locale("en", { firstDayOfWeek: "mon123" })
(unknown weekday but valid BCP-47)new Intl.Locale("en-u-fw-mon123")
(unknown weekday as BCP-47)
For each of these cases, we need to decide:
- What is the behavior of the constructor?
- What is the behavior of
Intl.Locale.prototype.firstDayOfWeek
?
Some options:
- RangeError in the constructor in both cases.
- Constructor succeeds; getter echoes back the string if it is not one of the well-known strings that gets converted to an integer
- Constructor succeeds; getter returns the default first day of week for the locale as an integer if the string is not one of the well-known strings
- We re-litigate the decision in #70 and always echo back the string
I don't see a reason for cases 1 and 2 to differ in behavior.
from proposal-intl-locale-info.
- Constructor succeeds; getter returns undefined if the string is not one of the well-known strings
from proposal-intl-locale-info.
I don't see a reason for cases 1 and 2 to differ in behavior.
agree
from proposal-intl-locale-info.
TG2 discussion: https://github.com/tc39/ecma402/blob/master/meetings/notes-2023-10-12.md#locale-with--u-fw--value-other-than-13-value-will-cause-assertion
Conclusion: We already have a getter for the integers in getWeekInfo().firstDay
, and anything we do here is going to break some consistency invariant. Therefore, TG2 is bringing a recommendation to TG1 to always return a string from Intl.Locale.prototype.firstDayOfWeek
.
from proposal-intl-locale-info.
I sent a private email to @littledan to explain why we like to change Intl.Locale.prototype.firstDayOfWeek to reuturn string or undefined from number or undefined. And how Intl.Locale.prototype.getWeekInfo().firstDay already address the use case he has in mind and will not be impacted by this.
from proposal-intl-locale-info.
I don't want to cause you all to spend too much more time on this. Although my intuition was option 1, this isn't something that I am going to insist on.
I just don't see the argument for why this API should be lenient, given that from a developer's perspective, they have to expect it to throw. As a non-expert in this area, I would've expected that getWeekInfo().firstDay === firstDayOfWeek
generally--the shape of the firstDayOfTheWeek
API makes it look more like something that's directly semantically meaningful, rather than a literal fragment of the input locale string. I don't think this API is as usable as it could be.
Nevertheless, I see that the current API is consistently lenient in exactly this way: e.g., you can get hourCycle
to be whatever you want as well, by passing the right string as the locale (but not as an option). Seems consistent to return a string from here, even if it's worse for JS developers. So I'm fine with ECMA-402 going that way, even if I'd prefer to go back in time and validate hc while returning a number here (calendars may be different because the set of calendars changes over time).
from proposal-intl-locale-info.
I don't want to cause you all to spend too much more time on this. Although my intuition was option 1, this isn't something that I am going to insist on.
I just don't see the argument for why this API should be lenient, given that from a developer's perspective, they have to expect it to throw. As a non-expert in this area, I would've expected that
getWeekInfo().firstDay === firstDayOfWeek
generally--the shape of thefirstDayOfTheWeek
API makes it look more like something that's directly semantically meaningful, rather than a literal fragment of the input locale string. I don't think this API is as usable as it could be.Nevertheless, I see that the current API is consistently lenient in exactly this way: e.g., you can get
hourCycle
to be whatever you want as well, by passing the right string as the locale (but not as an option). Seems consistent to return a string from here, even if it's worse for JS developers. So I'm fine with ECMA-402 going that way, even if I'd prefer to go back in time and validate hc while returning a number here (calendars may be different because the set of calendars changes over time).
Dear @littledan : I think we need to look at this as two layers of functionality of the API
Layer 1 - as an API for parsing the locale string and option bag resolution layer
Layer 2- as an API to get locale information
For the API in layer 2- number is for sure what is best- and that is address in getWeekInfo().firstDay
but we still have the need for the API in layer 1 - and that is what all the getter of the Intl.Locale did- those getter are returning the parsing + option bag resolution result. And that is operate in the space of Locale string therefore, should be kept in string. Otherwise, if the string has other value, it will be very difficult to return the result. One possible case to keep it in string is the future extension to support French Revolution Calendar- which have 10 days per week:
See https://en.wikipedia.org/wiki/French_Republican_calendar#Ten_days_of_the_week
I do not think there is a strong case to support such calendar right now and I do not believe CLDR has any plan to support that. But just an example to show you in the history there are people prefer a week is not 7 days.
from proposal-intl-locale-info.
Yes, I accept that this API has those two layers. I don’t think this separation does an ideal job serving its users, but it is possible to work through given the existence of the other layer. So it is OK to proceed as you proposed, from my perspective.
from proposal-intl-locale-info.
So... I plan to change the spec to
I. Intl.Locale.prototype.firstDayOfWeek returns: <<undefined, "mon", "tue", "wed", "thu", "fri", "sat", "sun">>
2. {firstDayOfWeek} for new Intl.Locale() accepts: <<undefined, "mon", "tue", "wed", "thu", "fri", "sat", "sun", "0", "1", "2", "3", "4", "5", "6", "7">> Default: undefined
where "0", "7", "sun" all repesent Sunday in option reading
from proposal-intl-locale-info.
Propose fix in #79
from proposal-intl-locale-info.
So... I plan to change the spec to
I. Intl.Locale.prototype.firstDayOfWeek returns: <<undefined, "mon", "tue", "wed", "thu", "fri", "sat", "sun">> 2. {firstDayOfWeek} for new Intl.Locale() accepts: <<undefined, "mon", "tue", "wed", "thu", "fri", "sat", "sun", "0", "1", "2", "3", "4", "5", "6", "7">> Default: undefined where "0", "7", "sun" all repesent Sunday in option reading
sorry, this is not right. I need to rework on this. Intl.Locale.prototype.firstDayOfWeek need to return string, but not just that 7 string values.
from proposal-intl-locale-info.
TG2 discussion: https://github.com/tc39/ecma402/blob/master/meetings/notes-2023-11-16.md#locale-with--u-fw--value-other-than-13-value-will-cause-assertion-78
from proposal-intl-locale-info.
This issue is addressed in #79
from proposal-intl-locale-info.
Related Issues (20)
- CharacterDirectionOfLocale doesn't handle top-to-bottom and bottom-to-top scripts HOT 1
- CharacterDirectionOfLocale doesn't take script and region subtags into account HOT 1
- Intl.Locale.prototype.textInfo is missing "lineOrder" information HOT 1
- "direction" property in Intl.Locale.prototype.textInfo is possibly ambiguous HOT 2
- textInfo.direction is a very simplified model of reality HOT 19
- The textInfo getter seems unwell HOT 5
- Reverse date creation HOT 1
- Add support for -u-fw and other Unicode extensions that could impact the result HOT 13
- `.getTimezones()` seems problematic HOT 3
- Consistent "region" subtag requirements for getTimeZones() and getWeekInfo()/getHourCycles() HOT 8
- Tc39 HOT 1
- Possible of addition of Intl.Locale.prototype.getCurrencies()? HOT 1
- Define fallback behaviour in {Calendars,Collations,HourCycles,NumberingSystems,CharacterDirection,WeekInfo}OfLocale HOT 6
- WeekInfoOfLocale need to convert loc.[[FirstDayOfWeek]] to number before return HOT 1
- Ensure getWeekInfo will not not listen to 'rg' HOT 3
- minimalDays HOT 2
- CharacterDirectionOfLocale: Confusing reference to "characterOrder"
- CharacterDirectionOfLocale: Define if locales are maximized before lookup
- CharacterDirectionOfLocale: Returns "ltr" for scripts without any character direction
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from proposal-intl-locale-info.