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

Translate composition-vs-inheritance #5

Merged

Conversation

Naturalclar
Copy link
Contributor

This PR is part of Issue #4

I've translated the composition-vs-inheritance document to Japanese, based on translation on crowdin.

@Naturalclar Naturalclar mentioned this pull request Jan 31, 2019
90 tasks
@smikitky
Copy link
Member

smikitky commented Feb 1, 2019

ありがとうございます!
LGTM on my smartphone, but please wait for a couple of hours before merging.

Copy link
Member

@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.

I reviewed and made comment on on point.
Please check and reply it!


それはそれとして、日本語訳のレビューは日本語の話も多くなると思うので、 PR へのレビューコメントは日本語ベースで良いかなぁと思ったりしていますが、みなさんどうでしょう。

content/docs/composition-vs-inheritance.md Outdated Show resolved Hide resolved
Copy link
Member

@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.

よさそうだと思います。対応ありがとうございます!

@smikitky
Copy link
Member

smikitky commented Feb 1, 2019

@potato4d 文末のコロンの有無についてご意見はありますか? 原文がコロンで終わっていても 以下の例を見てください: ではなく 以下の例を見てください。 のような形で訳した例が Crowdin には多いのですが、統一すべきでしょうか。特に強い信念でそうしたというより、単に昔の自分の好みがそうだったというだけなのですが…

@potato4d
Copy link
Member

potato4d commented Feb 1, 2019

@smikitky
これは好みによると思うので、これまでの翻訳実績を尊重していきましょうー。
逆に最後が : で終わるのを制限するルールは特にないので、最後が : で終わる場合は言い切るようにするルールを設けても良いと思います。

@smikitky smikitky self-assigned this Feb 1, 2019
@@ -80,13 +80,13 @@ function App() {

[**Try it on CodePen**](https://codepen.io/gaearon/pen/gwZOJp?editors=0010)

React elements like `<Contacts />` and `<Chat />` are just objects, so you can pass them as props like any other data. This approach may remind you of "slots" in other libraries but there are no limitations on what you can pass as props in React.
`<Contacts />` `<Chat />` のような React の要素はただのオブジェクトなので、他のあらゆるデータと同様に `props` として渡すことができます。
Copy link
Member

Choose a reason for hiding this comment

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

"This approach may remind you of “slots” in other libraries but there are no limitations on what you can pass as props in React." の訳が抜けています

Copy link
Contributor Author

Choose a reason for hiding this comment

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

見逃していました。該当部分を追記致しました。また、日本のエンジニアならVue, Angular以外にもRails(Riot.jsというケース) からくる人も多いと思うので、 slot に該当する yield も例として追加しました。

Copy link
Member

@smikitky smikitky left a comment

Choose a reason for hiding this comment

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

すみません、いくつか訳抜けや疑問点を発見したので対応をお願い致します。

@Naturalclar
Copy link
Contributor Author

レビューありがとうございます!
指摘箇所に対応致しました。確認お願いします。

@potato4d
Copy link
Member

potato4d commented Feb 1, 2019

@saitoeku3
今 master に TextLint が導入されたので、お手数ですが rebase もしくは merge をお願いしてもよろしいでしょうか 🙏

@Naturalclar
Copy link
Contributor Author

Text Lintかけて指摘箇所を直しました:pray:

@smikitky smikitky merged commit 7adb904 into reactjs:master Feb 1, 2019
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.

3 participants