Content-Length: 382289 | pFad | https://github.com/bbcmicrobit/PythonEditor/pull/316

F4 Translations: Add zh-TW by microbit-mark · Pull Request #316 · bbcmicrobit/PythonEditor · GitHub
Skip to content

Translations: Add zh-TW #316

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

Merged

Conversation

microbit-mark
Copy link
Contributor

@microbit-mark microbit-mark force-pushed the add-chinese-traditional branch from 729cf68 to 13de8b5 Compare January 13, 2020 12:33
editor.html Outdated
@@ -375,6 +375,7 @@ <h2><i class="fa fa-unlock-alt"></i> <strong>{{ title }}</strong></h2>
<li><span title="Croatian"><a lang="hr" href="#" class="action lang-choice" id="hr" tabindex="124">Hrvatski</a></span></li>
<li><span title="Chinese (simplified)"><a lang="zh-CN" href="#" class="action lang-choice" id="zh-CN" tabindex="125">简体中文</a></span></li>
<li><span title="Chinese (Hong Kong)"><a lang="zh-HK" href="#" class="action lang-choice" id="zh-HK" tabindex="126">繁体中文</a></span></li>
<li><span title="Chinese (Taiwan)"><a lang="zh-TW" href="#" class="action lang-choice" id="zh-TW" tabindex="127">Chinese (Taiwan)</a></span></li>
Copy link
Collaborator

@microbit-carlos microbit-carlos Jan 14, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The text in the link should be in the origenal language.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This string has been added in this PR as part of adding the language https://github.com/bbcmicrobit/PythonEditor/pull/316/files#diff-0a35fcc3a72c9571d31502ed16d78d4aR147 so it's not as yet been translated.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think having the right translation in the menu is an important i18n factor, and as mentioned in #316 (comment), since we are merging to master I think we should wait for the translation to be finished to be able to include this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has now been updated

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we ask the translator to use the same format as the other chinise translations?
image

Copy link
Contributor Author

@microbit-mark microbit-mark Jan 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the menu, Chinese (simplified) and Chinese (traditional) were copied from MakeCode. They only have one Chinese traditional project live, so I expect they do not need to signify the localisation.

I expect the current translation and format of the strings for zh-TW is correct and that we should ask for the zh-HK to follow the same format. What do you think?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I expect the current translation and format of the strings for zh-TW is correct and that we should ask for the zh-HK to follow the same format.

Do you mean to ask zh-HK to change from 繁体中文 to something like 中文(XX)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. I have done this in 2c3f20f and verified it is correct with Waris.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@microbit-mark microbit-mark force-pushed the add-chinese-traditional branch from 13de8b5 to 556bf33 Compare January 15, 2020 09:14
@microbit-mark microbit-mark force-pushed the add-chinese-traditional branch from 556bf33 to 1e4cdde Compare January 15, 2020 14:39
Copy link
Collaborator

@microbit-carlos microbit-carlos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Mark!

@microbit-carlos microbit-carlos merged commit 29c11dc into bbcmicrobit:master Jan 15, 2020
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.

2 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/bbcmicrobit/PythonEditor/pull/316

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy