Add Chinese Simplified localization file. #1593
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Found one new string that may needs localization ("The video is encrypted and we do not have the keys to decrypt it.").
Was not able to run test on my Windows 8.1 computer.
When run grunt, it complains:
Warning: Command failed: 'java' is not recognized as an internal or external command.
When run grunt test, it complains:
PhantomJS 1.9.7 (Windows 8) Player Minified should be able to access expected player API methods FAILED.
Although the tests are failed, I don't see the localization has anything to do with the error.
|
Thanks! The testing issue appears to be a lack of Java. It's required to minify the code using closure compiler. But you're right, it's not related to the Is 'chs' the right name for the file? We were just trying to figure this out in #1589. |
|
How about naming |
|
As @DevGavin mentioned in #1589, it looks like the language codes we should use are zh, zh-Hans, and zh-Hant. I'll go ahead and pull this in (which includes zh.json), but should we consider creating the other files as well? |
jgubman
added a commit
to jgubman/video.js
that referenced
this pull request
Oct 29, 2014
* 'stable' of /~https://github.com/videojs/video.js: (22 commits) Added line to changelog for 4.10.1 Release 4.10.1 Removed alert... Release 4.10.0 @heff turned on the custom html controls for touch devices. closes videojs#1617 updated options to include sans-children, and included self-hosted font blurb added CORS information to tracks guide. Fixes videojs#837 doc updates + readme quick start updated options guide to include components @heff Added the ability to set options for child components directly in the parent options. closes videojs#1599 @DevGavin added a Simplified Chinese translation. closes videojs#1593 @mmcc fixed an issue with the VolumeButton assuming it was vertical by default. closes videojs#1592 @heff enhanced the event listener API to allow for auto-cleanup of listeners on other componenets and elements. closes videojs#1588 @mmcc fixed an issue where errors on source tags could get missed. closes videojs#1575 Added doc for remaining time and removed onWaitEnd doc maybe actually check for keyLocation Update languages.md @heff updated the poster to use CSS styles to display; fixed the poster not showing if not originally set. closes videojs#1568 Release 4.9.1 Bumped to videojs-swf v4.5.1 to fix a data sanitization issue. closes videojs#1587 ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Found one new string that may needs localization ("The video is encrypted and we do not have the keys to decrypt it.").
Was not able to run test on my Windows 8.1 computer.
When run grunt, it complains:
Warning: Command failed: 'java' is not recognized as an internal or external command.
When run grunt test, it complains:
PhantomJS 1.9.7 (Windows 8) Player Minified should be able to access expected player API methods FAILED.
Although the tests are failed, I don't see the localization has anything to do with the error.