-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add Simplified Chinese translation of Quickopen command #240
Conversation
Thanks for the PR, @WaiSiuKei ! We will have a look at this as soon as we have the time to. |
Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, native speaker here. Overall this PR is very good, here is my review for some optional suggestions.
package.nls.zh-cn.json
Outdated
"python.command.python.buildWorkspaceSymbols.title": "构建工作区符号", | ||
"python.command.python.runtests.title": "运行全部单元测试", | ||
"python.command.python.debugtests.title": "调试全部单元测试", | ||
"python.command.python.execInTerminal.title": "在终端/命令行运行 Python 文件", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
在命令行运行 Python 文件
这样简洁一点。This removes the duplication of 终端/命令行 for simplicity.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes.
package.nls.zh-cn.json
Outdated
"python.command.python.runCurrentTestFile.title": "运行当前单元测试文件", | ||
"python.command.python.runFailedTests.title": "运行失败的单元测试", | ||
"python.command.python.execSelectionInTerminal.title": "在 Python 终端运行选定内容/行", | ||
"python.command.python.execSelectionInDjangoShell.title": "在 Django Shell 运行选定内容/行", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
用 Python 命令行执行选定内容
更简洁。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed these duplications.
package.nls.zh-cn.json
Outdated
@@ -0,0 +1,50 @@ | |||
{ | |||
"python.command.python.sortImports.title": "排序引入的模块", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
对 import 进行排序
会 python 用 vscode 的都能明白什么是 import.
Python developers and vscode users can understand what import
means here. No need to translate the keyword.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
package.nls.zh-cn.json
Outdated
@@ -0,0 +1,50 @@ | |||
{ | |||
"python.command.python.sortImports.title": "排序引入的模块", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's confused, how about 排序 import 语句
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
package.nls.zh-cn.json
Outdated
"python.command.python.sortImports.title": "排序引入的模块", | ||
"python.command.python.startREPL.title": "启动 REPL", | ||
"python.command.python.buildWorkspaceSymbols.title": "构建工作区符号", | ||
"python.command.python.runtests.title": "运行全部单元测试", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IHMO, 全部
-> 所有
is better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, VSCode used 91 '所有' and 35 '全部', in my local branch.
package.nls.zh-cn.json
Outdated
"python.command.python.selectAndRunTestFile.title": "运行单元测试文件...", | ||
"python.command.python.runCurrentTestFile.title": "运行当前单元测试文件", | ||
"python.command.python.runFailedTests.title": "运行失败的单元测试", | ||
"python.command.python.execSelectionInTerminal.title": "在 Python 终端运行选定内容/行", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about 在 Python 终端中运行选定内容/行
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as below ↓.
package.nls.zh-cn.json
Outdated
"python.command.python.runCurrentTestFile.title": "运行当前单元测试文件", | ||
"python.command.python.runFailedTests.title": "运行失败的单元测试", | ||
"python.command.python.execSelectionInTerminal.title": "在 Python 终端运行选定内容/行", | ||
"python.command.python.execSelectionInDjangoShell.title": "在 Django Shell 运行选定内容/行", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about 在 Django Shell 中运行选定内容/行
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '中' (means 'in') seems a little annoying, but it is widely used in the translation of VSCode, will add it.
package.nls.zh-cn.json
Outdated
"python.command.python.execSelectionInTerminal.title": "在 Python 终端运行选定内容/行", | ||
"python.command.python.execSelectionInDjangoShell.title": "在 Django Shell 运行选定内容/行", | ||
"python.command.jupyter.runSelectionLine.title": "运行选定内容/行", | ||
"python.command.jupyter.execCurrentCell.title": "运行 Cell", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about 运行当前 Cell
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, 'current' is not translated.
package.nls.zh-cn.json
Outdated
"python.command.python.execSelectionInDjangoShell.title": "在 Django Shell 运行选定内容/行", | ||
"python.command.jupyter.runSelectionLine.title": "运行选定内容/行", | ||
"python.command.jupyter.execCurrentCell.title": "运行 Cell", | ||
"python.command.jupyter.execCurrentCellAndAdvance.title": "运行 Cell 和 Advance", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Advance
mean move to the next Cell
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, fixed.
package.nls.zh-cn.json
Outdated
"python.snippet.launch.watson.label": "Python: Watson 应用", | ||
"python.snippet.launch.watson.description": "调试 Watson 应用", | ||
"python.snippet.launch.attach.label": "Python: 链接", | ||
"python.snippet.launch.attach.description": "链接远程调试器", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
attach
not mean 链接
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, MS translate it as '附加', https://msdn.microsoft.com/zh-cn/library/3s68z0b3.aspx
package.nls.zh-cn.json
Outdated
"python.command.python.selectAndRunTestFile.title": "运行单元测试文件...", | ||
"python.command.python.runCurrentTestFile.title": "运行当前单元测试文件", | ||
"python.command.python.runFailedTests.title": "运行失败的单元测试", | ||
"python.command.python.execSelectionInTerminal.title": "在 Python 终端中运行选定内容", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IHMO, missing Line
will let this is confused.
在 Python 终端中运行选定内容
don't include Run Line in Python Terminal
, because there may nothing been selected when Run Line
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, there is Line
in EN.
@DonJayamanne does the current failure have absolutely anything to do with the PR? I don't think it does. |
Thanks @mozillazg and @lambdaq for the reviews! If you have a chance, please see the changes @WaiSiuKei made and let us know if they address your concerns. |
package.nls.zh-cn.json
Outdated
@@ -0,0 +1,50 @@ | |||
{ | |||
"python.command.python.sortImports.title": "对 import 进行排序", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IHMO, 排序 import 语句
is better than 对 import 进行排序
. Because all other translations of command begin with a verb, only this is exceptional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@WaiSiuKei Good job! Thanks for your translation!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
# Conflicts: # package.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, why github is not diffing this commit?
Thanks for the reviews @mozillazg and @lambdaq ! @WaiSiuKei we will do a review to make sure the stuff is formatting appropriately, etc. and then we should be able to merge! |
Thanks everyone for helping make this happen! |
* upstream/master: Add Simplified Chinese translation of commands (#240)
thanks for ur hard work. |
@wangzhaohe we use the locale specified by VS Code, so you will have to change your locale there and the extension will pick it up. |
Add Simplified Chinese translation for Quickopen command.
Here is the screenshot: