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 API Reference of React.Component #127

Merged

Conversation

lightnet328
Copy link
Contributor

This PR is a part of #4.
Translate: React.Component – React

@lightnet328 lightnet328 changed the title doc: Translate API Reference of React.Component Translate API Reference of React.Component Feb 16, 2019
@smikitky smikitky self-requested a review February 17, 2019 05:35
@tesseralis tesseralis mentioned this pull request Feb 18, 2019
90 tasks
@netlify
Copy link

netlify bot commented Feb 20, 2019

Deploy preview for ja-reactjs ready!

Built with commit 5c79337

https://deploy-preview-127--ja-reactjs.netlify.com

@koba04 koba04 added the 🚫 in intial review 初期レビューの真っ最中 label Feb 20, 2019
Copy link
Member

@koba04 koba04 left a comment

Choose a reason for hiding this comment

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

@lightnet328
翻訳お疲れ様です!
長いページなので、コメント多くなってしまいましたがご確認お願いします! 🙇
細かいものばかりです。

content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved

Normally you should try to avoid all uses of `forceUpdate()` and only read from `this.props` and `this.state` in `render()`.
通常は `forceUpdate()` の使用をすべて避け、`render()` の `this.props` `this.state` からのみ読み取るようにしてください。
Copy link
Member

Choose a reason for hiding this comment

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

言い回しを調整しました [IMO]

Suggested change
通常は `forceUpdate()` の使用をすべて避け`render()``this.props``this.state` からのみ読み取るようにしてください。
通常は全ての `forceUpdate()` の使用を避け`render()``this.props``this.state` からのみ読み取るようにしてください。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちら、ご提案いただいた内容のほうが良いと思うので取り込ませていただきました。
また、「通常は forceUpdate() を使用せずに、」ではどうでしょうか?意訳しすぎでしょうか。

Copy link
Member

Choose a reason for hiding this comment

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

@lightnet328

また、「通常は forceUpdate() を使用せずに、」ではどうでしょうか?意訳しすぎでしょうか。

いいと思います!

content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
@koba04 koba04 removed the 🚫 in intial review 初期レビューの真っ最中 label Feb 20, 2019
koba04 and others added 13 commits February 21, 2019 01:32
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
@lightnet328
Copy link
Contributor Author

@koba04 レビューしていただき、ありがとうございます!
訳の誤りの修正と言い回しに関して、提案していただいたものの方が良いと思ったので取り込ませていただきました(1つ1つ修正の内容・理由について示していただき、ありがとうございます🙏)。
非常に細かい点について、1つコメントしたのでご確認いただきたいです。
#127 (comment)
よろしくお願いいたします。

Copy link
Member

@koba04 koba04 left a comment

Choose a reason for hiding this comment

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

@lightnet328 表示がおかしくなっている部分があったのでコメントしましたのでそこだけ確認お願いします 🙏
1つは自分がsuggestionした部分でした。失礼しました 🙇

content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
@lightnet328 lightnet328 force-pushed the translate/api-reference-react-component branch from 2c70307 to 80d8236 Compare February 22, 2019 13:51
koba04 and others added 13 commits February 22, 2019 23:00
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
Co-Authored-By: lightnet328 <lightnet328@gmail.com>
@lightnet328 lightnet328 force-pushed the translate/api-reference-react-component branch from 7763181 to bb7b035 Compare February 22, 2019 14:00
@lightnet328
Copy link
Contributor Author

Markdownが崩れている部分について修正しました🙇

@koba04
Copy link
Member

koba04 commented Feb 22, 2019

@lightnet328 対応ありがとうございました!

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.

良さそうです!
ありがとうございました!

@potato4d potato4d merged commit 671a685 into reactjs:master Feb 23, 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.

None yet

4 participants