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

Add localization for Japanese #434

Merged
merged 5 commits into from
Dec 19, 2017

Conversation

YuichiNukiyama
Copy link

Basically it matches Japanese translation of vs code.

@brettcannon
Copy link
Member

Thank you for the PR, @YuichiNukiyama ! I have asked on Twitter for verification of your translation since no one on the team speaks Japanese. 😄

@mpenkov
Copy link

mpenkov commented Dec 16, 2017

JA speaker here, had a look, LGTM with one question: why was Russian added in the README? Is this a relevant change?

@YuichiNukiyama
Copy link
Author

Thank you for your review. I remove ru from readme.

Copy link

@yuja yuja left a comment

Choose a reason for hiding this comment

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

Native speaker here. Looks good but for "Run Cell and Advance".

"python.command.python.execSelectionInDjangoShell.title": "Django シェルで選択範囲/行を実行",
"python.command.jupyter.runSelectionLine.title": "選択範囲/行を実行",
"python.command.jupyter.execCurrentCell.title": "セルを実行",
"python.command.jupyter.execCurrentCellAndAdvance.title": "セルとアドバンスを実行",
Copy link

Choose a reason for hiding this comment

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

Here "Advance" is a verb. Perhaps this line should be something like
"セルを実行して次へ".

@@ -0,0 +1,50 @@
{
"python.command.python.sortImports.title": "Imports を並び替える",
Copy link

Choose a reason for hiding this comment

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

"Import 文を並び替える" seems slightly better.
A plural form isn't common in Japanese.

{
"python.command.python.sortImports.title": "Imports を並び替える",
"python.command.python.startREPL.title": "REPL を開始",
"python.command.python.buildWorkspaceSymbols.title": "ワークスペース シンボルをビルド",
Copy link

Choose a reason for hiding this comment

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

Maybe it's better to say "ワークスペースの" for adjective "Workspace".

"python.snippet.launch.attach.label": "Python: アタッチ",
"python.snippet.launch.attach.description": "リモートデバッグのためにデバッガをアタッチ",
"python.snippet.launch.scrapy.label": "Python: Scrapy",
"python.snippet.launch.scrapy.description": "統合ターミナル/コンソールで Scrapy"
Copy link

Choose a reason for hiding this comment

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

No idea if "Scrapy" as a verb can be recognized by ordinary people
in Japan. Maybe it could be "Scrapy を実行" or "Scrapy る" ?

@YuichiNukiyama
Copy link
Author

Thaks @yuja ! I follow your advice

Copy link

@yuja yuja left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@DonJayamanne
Copy link

@brettcannon as this has been reviewed by two other individuals, I vote to merge this PR. Will merge later today.

@DonJayamanne DonJayamanne merged commit 08008f3 into microsoft:master Dec 19, 2017
@YuichiNukiyama YuichiNukiyama deleted the localize_ja branch December 20, 2017 06:18
DonJayamanne added a commit that referenced this pull request Jan 9, 2018
* upstream/master:
  Release 0.9.1 (#458)
  Add localization for Japanese (#434)
  Add Svn to uriSchemesToIgnore (#443)
@brettcannon brettcannon mentioned this pull request Jan 29, 2018
38 tasks
@lock lock bot locked as resolved and limited conversation to collaborators Jul 31, 2019
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.

5 participants