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

Avoid using emoji as a collation example #15645

Merged
merged 1 commit into from
May 4, 2022

Conversation

hsivonen
Copy link
Contributor

@hsivonen hsivonen commented May 3, 2022

Summary

Instead of using emoji as an example of an explicitly-requested collation type, use zhuyin instead.

Motivation

The emoji collation type is intended for charts and emoji pickers and that it combines with a language like English is incidental and works only because English is an alias for the root collation. Notably, it doesn't combine with languages that aren't aliases for the root.

The non-default collation types that have the most present-day use case relevance are probably zhuyin for Traditional Chinese and phonebk for German and of these, phonebooks are arguably more special-casey. Therefore, using zhuyin as the example.

Supporting details

tc39/proposal-intl-locale-info#33 (comment)

Related issues

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

@hsivonen hsivonen requested a review from a team as a code owner May 3, 2022 09:58
@hsivonen hsivonen requested review from teoli2003 and removed request for a team May 3, 2022 09:58
Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

👍 thanks @hsivonen !

@wbamberg wbamberg merged commit 2fce978 into mdn:main May 4, 2022
@hsivonen hsivonen deleted the collationexample branch May 5, 2022 06:01
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

4 participants