Skip to content

Conversation

@thinca
Copy link
Member

@thinca thinca commented Feb 1, 2016

#164

レビューお願いします!

doc/repeat.jax Outdated
Copy link
Member

Choose a reason for hiding this comment

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

レコーディング中にその最中に へ修正してください。
同じ単語が近くに並ぶと読みにくいので。

Copy link
Member Author

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 2, 2016

私からは以上、指摘3点、単なる雑談1件の計4点です。

@thinca
Copy link
Member Author

thinca commented Feb 2, 2016

指摘された部分と、ついでに例示された部分の一部を日本語リソースでの表示に合わせて修正しました。

Copy link
Member Author

Choose a reason for hiding this comment

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

ついでに例示された部分の一部を日本語リソースでの表示に合わせて修正しました。

ここです。

Copy link

Choose a reason for hiding this comment

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

(Vim用語が全然わかっていませんが...)

現在いるフレーム -> 現在のフレーム

で良いのではないでしょうか ?

Copy link
Member Author

Choose a reason for hiding this comment

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

この辺りに特に Vim 特有の用語はないです。
最初は私も 現在のフレーム にしていたのですが、現在いるフレーム の方がデバッガに慣れていない人にもわかりやすいかな、と思い、このようにしてみました。
現在のフレーム の方が良い、と言う意見が他にもあれば変えます。

@koron
Copy link
Member

koron commented Feb 2, 2016

細かい議論は今後もあると思いますが、訳がそろってますしいったんマージします。
残課題はその都度issueかPRか直接修正でやりましょう。

koron added a commit that referenced this pull request Feb 2, 2016
Translate some remaining hunks in repeat.jax
@koron koron merged commit fce2b5f into devel Feb 2, 2016
@koron koron deleted the repeat branch February 2, 2016 14:49
@koron
Copy link
Member

koron commented Feb 2, 2016

thx!

@thinca
Copy link
Member Author

thinca commented Feb 2, 2016

🎉

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.

4 participants