Closed aphillips closed 3 years ago
Looks good to me.
Here are some commments: 1) Not part of this pull request, but the link to CLDR https://cldr.unicode.org/ from the "references" section does not work for me. http://cldr.unicode.org/ works (although HTTPS would be better). 2) Reference to CLDR as a source for locale processing rules, e.g. here: "A language tag that follows the additional processing rules defined by [CLDR]" Is there a list of the locale processing rules available, e.g. as part of LDML? Pointing to such a list would be helpful for users & implementers of CLDR. If there is no such list, maybe put it into the LTLI document?
The rest of the pull request looks good to me. You may also want to merge the pull request and address these comments in a new pull request.
@fsasaki
On (1), yikes! You're correct. I guess I just assumed Unicode had moved to https
.
On (2), I agree. The rules and such are here and I'll add the link to our doc.
Ugh. You're all officially going to hate me.
I'm going to merge this now, because in the course of addressing @fsasaki's #2, I ended up doing a fairly significant reorg.
Minor textual edits to definitions for Locale, Unicode Locale, and the locale-netural example. I slightly tweaked @fsasaki's text where my previous suggestion made the text repetitive.
Preview | Diff