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 "hydrate" #653

Merged
merged 7 commits into from
Sep 4, 2023
Merged

Translate "hydrate" #653

merged 7 commits into from
Sep 4, 2023

Conversation

riku929hr
Copy link
Contributor

react-dom: APIshydrate の翻訳を行いました。

行ったことは以下です。

こちら で宣言しておりましたが、もし先にご対応されているようでしたら、閉じていただいて構いません。

よろしくお願いいたします。

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.

ありがとうございます。レビューを行いましたので確認をお願いします。

src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved

* `reactNode`: The "React node" used to render the existing HTML. This will usually be a piece of JSX like `<App />` which was rendered with a `ReactDOM Server` method such as `renderToString(<App />)` in React 17.
* `reactNode`: 既存の HTML をレンダーするのに使用される「React ノード」。これは通常、React 17 における `ReactDOM Server` のメソッド(例:`renderToString(<App />)`)でレンダーされた JSX の一部である`<App />`のようなものです。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* `reactNode`: 既存の HTML をレンダーするのに使用される「React ノード」。これは通常、React 17 における `ReactDOM Server` のメソッド(例:`renderToString(<App />)`)でレンダーされた JSX の一部である`<App />`のようなものです。
* `reactNode`: HTML を事前レンダーするのに使用された「React ノード」。これは通常、React 17 における `ReactDOM Server` のメソッド(例:`renderToString(<App />)`)でレンダーされた JSX である `<App />` のようなものです。

この "piece of" は英文法的に必要なゴミみたいなものですので日本語では訳さないでいいかなと思います

Copy link
Member

@smikitky smikitky Oct 13, 2023

Choose a reason for hiding this comment

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

@riku929hr 偶然これを見つけたのでお答えしますと、この "piece of" は英語で不可算名詞を無理やり数えるときの表現です。受験英語的には "a piece of paper", "a piece of information" が有名で、特に後者は「情報の一部」ではなく単に「情報」とするか、「1 件の情報」とすることになると思います。

React ドキュメントの場合 JSX や state は不可算名詞扱いなので、「ひと固まりの JSX」「複数の state ではなく 1 個の state」などと言いたい文脈で "piece of" がよく出てきます。日本語の場合、無理やり "JSX 素片" などとすることもできるとは思いますが、単に無視するのが最も楽かな、ということでした。

src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
@riku929hr
Copy link
Contributor Author

riku929hr commented Aug 21, 2023

@smikitky
とても丁寧にレビューしていただきありがとうございます。
イージーな和訳ミスも数か所あったことに気づきました。お手数おかけしました。

修正したので、再度ご確認いただけますでしょうか。よろしくお願いいたします。

@riku929hr riku929hr requested a review from smikitky August 21, 2023 12:54
@github-actions
Copy link

github-actions bot commented Aug 22, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

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.

ありがとうございました!
AIもまだミスが多いので人間が頑張らないとですね…

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.

すみません、前指摘したものに関して幾つか修正残しがあるようです。
加えて新たに気付いた問題もいくつかあり、対応願えますでしょうか。

src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
src/content/reference/react-dom/hydrate.md Outdated Show resolved Hide resolved
@riku929hr
Copy link
Contributor Author

@smikitky
ご指摘ありがとうございます。お手数おかけしすみません。
修正しましたのでご確認お願いいたします。

@smikitky
Copy link
Member

smikitky commented Aug 23, 2023

お手数をおかかけしています。
3 箇所 suggestion が残っていたのでこちらでコミットさせていただきました。(cfadc24)

@riku929hr
Copy link
Contributor Author

すみません、ありがとうございました🙇

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.

👍 ありがとうございます!!!

@koba04 koba04 merged commit 5b8750e into reactjs:main Sep 4, 2023
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