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

fix(UI): Fix broken language names on Google TV #5613

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

joeyparrish
Copy link
Member

@joeyparrish joeyparrish commented Sep 6, 2023

Reverts #5365: "feat(UI): Use Intl.DisplayNames to show the language name"

On the Google TV Chromecast just added to the lab, Intl.DisplayNames is returning the input as output again for some languages. For example, instead of en => English, we get en => en. Same for fr. Oddly, it works for some others, such as zh.

We can follow up with a change that either avoids Google TV or that falls back to the internal map when Intl.DisplayNames returns its input as output.

@shaka-bot
Copy link
Collaborator

Incremental code coverage: No instrumented code was changed.

@joeyparrish joeyparrish merged commit fc93292 into main Sep 6, 2023
21 of 22 checks passed
@joeyparrish joeyparrish deleted the revert-5365-international-languages branch September 6, 2023 22:04
joeyparrish added a commit that referenced this pull request Sep 8, 2023
Reverts #5365:  "feat(UI): Use Intl.DisplayNames to show the language name"

On the Google TV Chromecast just added to the lab, Intl.DisplayNames is returning the input as output again for some languages.  For example, instead of `en` => `English`, we get `en` => `en`.  Same for `fr`.  Oddly, it works for some others, such as `zh`.

We can follow up with a change that either avoids Google TV or that falls back to the internal map when Intl.DisplayNames returns its input as output.
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Nov 5, 2023
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Nov 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants