Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

fixed #901 , mist now get best matched language code #998

Merged

Conversation

ChihChengLiang
Copy link
Contributor

hi~
I fixed #901 by adding a getBestMatchedLangCode function in i18n module. The mist now get the closest language code from supported language.

Better test this fix with a computer whose default language is en-US.

@luclu
Copy link
Contributor

luclu commented Jul 19, 2016

Awesome @ChihChengLiang, changing works again (for some languages, still no German for me ;)). Please label this PR as in progress/please review.

@ChihChengLiang
Copy link
Contributor Author

Oops, too bad it seems I don't have permission to add a label. Would you please do it for me?
I'll check why German don't work.

@ChihChengLiang
Copy link
Contributor Author

ChihChengLiang commented Jul 19, 2016

Thanks @luclu .
I found that TAPi18n.setLanguage("de") does not behave normally, compared to other languages. It'll take me some time to dive deeper to the TAPi18n code. Let's create an issue for this: #1000

@lock
Copy link

lock bot commented Mar 31, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Mar 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[i18n] Default language button change wallet language to zh, in a zh-TW locale computer.
3 participants