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.

Sign up for GitHub

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

Jump to bottom

Translate doc/pi_netrw.jax #207

Merged
merged 6 commits into from
Feb 12, 2016
Merged

Translate doc/pi_netrw.jax #207

merged 6 commits into from
Feb 12, 2016

Conversation

tyru
Copy link
Member

@tyru tyru commented Feb 10, 2016

via #164

tyru added 5 commits February 9, 2016 00:19
@tyru
Translate pi_netrw.jax (WIP)
b88a3a4
@tyru
Update
ccfa1b7 
* `*netrw-usermaps*`
* `*netrw-p17*`
* `*netrw-P18*`
* `*netrw-P19*`
* `*netrw-P20*`
* `*netrw-P21*`
* `*netrw-P22*`
@tyru
Translated *netrw-debug*
1601cad
@tyru
Translate pi_netrw.jax
d8a6e4d 
1. According to `git diff f8de019^ f8de019 -- doc/pi_netrw.jax`, check if
   there are any untranslated texts
2. I removed `>` at the beginning of code blocks wrongly,
   so check horizontally seeing 1's diff
3. Translate `(remove NOSPAM from Campbell's email first)`
   at the beginning of file
@tyru
Translate pi_netrw.jax
497c74d 
1. Use terms of respect checking my commits again
2. In TOC or related topics and so on, terms of respect are not used
@tyru tyru mentioned this pull request Feb 10, 2016
Closed
31 tasks
@koron
Copy link
Member

koron commented Feb 11, 2016

コイツはでかいので、明らかな間違い以外は無視してマージして、
それからちょこちょこPRもしくは直接commitで直していくことにしましょ。

もうちょっとしたら見ます。

@tyru
Copy link
Member Author

tyru commented Feb 11, 2016

了解です。
ありがとうございます！

koron
koron reviewed Feb 11, 2016
View reviewed changes
doc/pi_netrw.jax
@@ -4,7 +4,7 @@
NETRW REFERENCE MANUAL by Charles E. Campbell
------------------------------------------------
Author: Charles E. Campbell <[email protected]>
(remove NOSPAM from Campbell's email first)
(まずメールアドレスから NOSPAM を削除してください)
Copy link
Member

Choose a reason for hiding this comment

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

現時点では修正不要だが: まず は要らなさそう。

@koron
Copy link
Member

koron commented Feb 11, 2016

@tyru とりあえず、私の方からは以上です。

全体的に、変な位置で改行を入れているのが気になりました。
正確には文の切れ目とか、翻訳がしやすくなるように入れているのだろうと理解しましたが、
読みやすさの点で言うと、日本語の場合なるべく右端(改行位置)が揃っている方が良いです。

今直すかは微妙なところですが、
何箇所かは翻訳とは関係ないところで改行を追加しているようですので、
それらについては戻してください。

@tyru
Copy link
Member Author

tyru commented Feb 11, 2016

現時点では修正不要だが: と付けてくれたコメント以外は修正しました。
あと改行の件はここ以外にないと思っているのですが、認識合ってますでしょうか？
https://github.com/vim-jp/vimdoc-ja/pull/207/files#r52606511

@koron
Copy link
Member

koron commented Feb 11, 2016

改行の件は、ちゃんとはチェックしていません。

翻訳したところで改行がおかしい箇所が多いので、途中から(この件に関しては)ちゃんと見ていません。
それらは、対応しなくても良いと先に書きましたが、読みやすさの観点からは現時点でやったほうが良いかもです。

@tyru
Copy link
Member Author

tyru commented Feb 11, 2016

あ、なぜか自分が既存の翻訳分で改行を入れてしまった部分のみの事を言ってると勘違いしてました。
すみません。了解です。

@koron
Copy link
Member

koron commented Feb 12, 2016

まぁ、許容しがたいわけではないので、他に意見がでてこないようならば、明朝(13日)にはマージします。

@tyru
Copy link
Member Author

tyru commented Feb 12, 2016

ありがとうございます。
改行の件はキャッチアップ後にぼちぼち修正していきます。

@koron
Copy link
Member

koron commented Feb 12, 2016

えーい。もういいや、みんなそろそろ寝ちゃってるだろうから、少し早いですけど新たな指摘は無いものとしてマージします。

koron added a commit that referenced this pull request Feb 12, 2016
@koron
Merge pull request #207 from vim-jp/pi_netrw
7237f82 
Translate doc/pi_netrw.jax
@koron koron merged commit 7237f82 into devel Feb 12, 2016
@koron koron deleted the pi_netrw branch February 12, 2016 16:33
@tyru
Copy link
Member Author

tyru commented Feb 12, 2016

ありがとうございます！

@tyru tyru mentioned this pull request Feb 13, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tyru @koron