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

Chinese localization #691

Closed
wants to merge 22 commits into from
Closed

Chinese localization #691

wants to merge 22 commits into from

Conversation

ztsai
Copy link

@ztsai ztsai commented Mar 14, 2014

  • Controls
  • Sprites
  • Music
  • Sound
  • Arrays
  • Boolean
  • Canvas
  • Color @yiting
  • Images
  • Math
  • Random
  • Vectors
  • Objects
  • Strings: @whleung
  • Paths
  • Points
  • Rects
  • Sensing
  • Motion
  • Shapes
  • Geolocation
  • Date
  • Sizes
  • Text
  • Matrix

Related to #690.

@ztsai ztsai closed this Mar 15, 2014
@ztsai ztsai deleted the 690_chinese_localization branch March 15, 2014 23:43
@ztsai
Copy link
Author

ztsai commented Mar 15, 2014

sorry, I screwed up the branch by using rebase when I should use merge.
i'll fix the branch and reopen the pull request

@ztsai ztsai reopened this Mar 15, 2014
rename locale directory to zh
and recopy json files to include sectionkey field
@ztsai
Copy link
Author

ztsai commented Mar 16, 2014

oh, right now only 2 character code is supported, so I changed it to zh for now
If @tyhoff adds support for 4 character code in the future, then we can do zh-tw

@CelticMinstrel
Copy link
Contributor

I know the docs said it needs a 2-character code, but is that actually true? Does the implementation check the length of the locale code anywhere?

@ztsai
Copy link
Author

ztsai commented Mar 16, 2014

yeah, in initLanguageFiles, it extracts the first 2 characters
var locale = (navigator.userLanguage || navigator.language || "en-US").substring(0,2);

@dethe
Copy link
Member

dethe commented Feb 7, 2015

I hate to close this, but our strategy for localization has changed pretty radically. On the plus side, future localizations should be much easier.

@dethe dethe closed this Feb 7, 2015
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.

4 participants