Skip to content
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

Rewrite of Unicode locale section based on telecon of 2020-10-01 #27

Merged
merged 3 commits into from Oct 2, 2020

Conversation

aphillips
Copy link
Contributor

@aphillips aphillips commented Oct 1, 2020

Merge changes from w3c
Rewrote the section on Unicode locale identifiers.
@aphillips aphillips requested review from fsasaki and r12a October 1, 2020 16:31
@r12a
Copy link
Contributor

r12a commented Oct 1, 2020

This looks much better to me. I'll make a couple of nit-notes against the source.

index.html Outdated
<p>A Unicode locale provides the ability to specify in a <a>language tag</a> some <a>international preference</a> variations that go beyond linguistic or regional variation or to select formatting behavior or content when there are multiple options or user preferences within a given locale. Unicode locale identifiers are well-formed [[BCP47]] language tags. [[CLDR]] also specifies some additional rules about the structure and content of the Unicode Locale's language tag as well as supplying specific interpretation of certain subtags. See <a href="https://www.unicode.org/reports/tr35/tr35.html#Unicode_locale_identifier">Section 3.2</a> of [[LDML]] for details. </p>
<p class="definition"><dfn data-lt="unicode locale|unicode locale identifier|unicode locale identifiers|unicode locales">Unicode Locale Identifier</dfn> or <em>Unicode Locale</em>. A <a>language tag</a> that follows the additional rules and restrictions on subtag choice defined in UTR#35 [[LDML]]. Any valid Unicode locale identifier is also a <a>valid</a> [[BCP47]] <a>language tag</a>, but a few <a>valid</a> <a>language tags</a> are not also valid Unicode locale identifiers.</p>
<p>[[CLDR]] defines and maintains two language tag extensions ([[RFC6067]] and [[RFC6497]]) that are related to <a>Unicode locale identifiers</a>. These extensions allow a <a>language tag</a> to express some <a>international preference</a> variations that go beyond linguistic or regional variation or to select formatting behavior or content when there are multiple options or user preferences within a given locale. <a>Unicode locale identifiers</a> are not required to include these extensions: they are only used when the locale being identified requires additional tailoring provided by one of these extensions. [[CLDR]] also applies specific interpretation of certain subtags when used as a locale identifier. See <a href="https://www.unicode.org/reports/tr35/tr35.html#Unicode_locale_identifier">Section 3.2</a> of [[LDML]] for details.</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at least as far as the extension, the rfcs are also related to BCP47 language tags, but perhaps we could say 'which are particularly relevant to Unicode locale identifiers, and define the sequence of tags that follow a -u or -t extension" ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You haven't explained the meaning of "but a few valid language tags are not also valid Unicode locale identifiers". Perhaps a single sentence para could be include here to say something like "Unicode locale identifiers come with a few rules about choice of subtags in a language tag: for example, deprecated subtags should be replaced with non-deprecated alternatives."

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By the way, that "but a few valid language tags are not also valid Unicode locale identifiers" is very helpful. Nice addition!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The RFCs are not just "related to BCP47 language tag". They are valid parts of language tags--part of BCP47--via the extension mechanism.

Your second comment is very helpful. See if these changes help.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

index.html Outdated

<p>Unicode's [[CLDR]] project maintains both of the [[BCP47]] extensions related to Unicode locales. The Unicode locale language tag extension [[RFC6067]] uses the <code>-u-</code> subtag, and provides subtags for selecting different locale-based formats and behaviors. See <a href="https://www.unicode.org/reports/tr35/tr35.html#Locale_Extension_Key_and_Type_Data">Section 3.6</a> of [[LDML]] for details.</p>
<p>The <strong>Unicode locale language tag extension</strong> [[RFC6067]] uses the <code>-u-</code> subtag, and provides subtags for selecting different locale-based formats and behaviors. See <a href="https://www.unicode.org/reports/tr35/tr35.html#Locale_Extension_Key_and_Type_Data">Section 3.6</a> of [[LDML]] for details.</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Unicode locale language tag extension" is rather long. Are you sure it's not just "Unicode locale extension"? Just asking.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I failed to link "language tag extension". My intention was to help remind folks that these are language tag extensions (valid in any language tag, not just Unicode locales). Then I failed to make the second one consistent with the first one... 😛.

See if the edit helps?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does.

@r12a r12a merged commit 89b2e51 into w3c:gh-pages Oct 2, 2020
@r12a
Copy link
Contributor

r12a commented Oct 2, 2020

Merged so that we can move on.

aphillips added a commit to aphillips/ltli that referenced this pull request Oct 2, 2020
Merge pull request w3c#27 from aphillips/gh-pages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants