Skip to content

update ja docs #2094

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
merged 5 commits into from
Mar 8, 2018
Merged

update ja docs #2094

merged 5 commits into from
Mar 8, 2018

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented Mar 8, 2018

/ping @re-fort @potato4d @lmiller1990 @38elements
どなたかレビューしてもらえると助かります! 🙏

Copy link

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

@kazupon
翻訳ありがとうございます!こちら一点提案だけコメントしておきました!


_ここではネストされたビューコンポーネントは省略されていますが、上記例の完全なソースコードを[ここ](https://jsfiddle.net/posva/22wgksa3/)で見つけることができます_

それから、このルート設定で上記のレイアウトを達成することができます:
Copy link

Choose a reason for hiding this comment

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

ここの「達成」ですが、確かに achieve をそのまま訳すとすると正しいのですが、あまり日本語として読みやすくない気がします。

ですので、「その後、このルート設定で上記のレイアウトを表現できることができます」のほうが良さそうかなーと思いますがどうでしょうか。
(日本語だと「下記の」と書けるところが原文が「この」になっているのも相まってなかなか複雑になってしまいますね 😅)

Copy link

@ykhirao ykhirao left a comment

Choose a reason for hiding this comment

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

@kazupon
翻訳お疲れ様です!ありがとうございます!:+1:
こちら簡単なニュアンスの部分だけコメントしました。
お手すきでご確認ください。

</div>
```

_ここではネストされたビューコンポーネントは省略されていますが、上記例の完全なソースコードを[ここ](https://jsfiddle.net/posva/22wgksa3/)で見つけることができます_
Copy link

@ykhirao ykhirao Mar 8, 2018

Choose a reason for hiding this comment

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

ここの「見つけることができます」ですが、日本語ですと「見ることができます」くらいのほうがニュアンスとしてわかりやすそうですが、いかがでしょうか。

英語だと「you can find 」の部分です。

}
```

この例の動作するデモは、[ここ](https://jsfiddle.net/posva/22wgksa3/)に見つけることができます。
Copy link

Choose a reason for hiding this comment

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

上記コメントと同様の部分になります。

@kazupon
Copy link
Member Author

kazupon commented Mar 8, 2018

@potato4d @ykhirao レビューありがとうございました! 🙏
指摘 👀 して頂いた部分、対応しましたので、再度お願いします!

Copy link

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

@kazupon 対応ありがとうございます!
良さそうです!

Copy link

@ykhirao ykhirao left a comment

Choose a reason for hiding this comment

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

@kazupon ご対応ありがとうございます!良さそうです:+1:

@kazupon
Copy link
Member Author

kazupon commented Mar 8, 2018

@potato4d @ykhirao
Thank you for your reiewing 😉

@kazupon kazupon merged commit 4a9470e into vuejs:dev Mar 8, 2018
@kazupon kazupon deleted the docs/ja branch March 8, 2018 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants