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: Close #187795, Language pack files for some extensions not found #187796

Closed
wants to merge 4 commits into from

Conversation

yiliang114
Copy link
Contributor

@yiliang114 yiliang114 commented Jul 13, 2023

To resolve an issue where some extended language packs fail to load
image

@yiliang114
Copy link
Contributor Author

Hi @TylerLeonhardt @bpasero Do you take up a little time to review the code for me?

@TylerLeonhardt
Copy link
Member

I would like to understand where the upper case CN is coming from... is that the browser? Does navigator.language in the Dev Tools Console of your browser return zh-CN?

@yiliang114
Copy link
Contributor Author

I would like to understand where the upper case CN is coming from... is that the browser? Does navigator.language in the Dev Tools Console of your browser return zh-CN?

Yes. console of chrome safari and firefox respectively

image

@yiliang114
Copy link
Contributor Author

It looks like there are only a few examples on MDN, not a complete list of languages.

However, there are relevant language codes in the chromium source code which may be used as a reference.

@TylerLeonhardt
Copy link
Member

Closing this as this should be fixed now. See: #187795 (comment)

@github-actions github-actions bot locked and limited conversation to collaborators Sep 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants