Conversation

riku929hr

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

行ったことは以下です。

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

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

Choose a reason for hiding this comment

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

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


* `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 素片" などとすることもできるとは思いますが、単に無視するのが最も楽かな、ということでした。

@riku929hr

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

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

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

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! 🙌

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

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

@riku929hr

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

@smikitky

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

@riku929hr

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

Choose a reason for hiding this comment

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

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

@koba04koba04 merged commit 5b8750e into reactjs:main Sep 4, 2023
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.