Skip to content

Translate "useCallback" hook reference#591

Merged
koba04 merged 8 commits intoreactjs:mainfrom
seckie:api_reference_use_callback
Jun 16, 2023
Merged

Translate "useCallback" hook reference#591
koba04 merged 8 commits intoreactjs:mainfrom
seckie:api_reference_use_callback

Conversation

@seckie
Copy link
Contributor

@seckie seckie commented May 21, 2023 •
edited
Loading

#573 に引き続いて2つ目の翻訳です。

@seckie seckie changed the title Translate useCallback on "Reference" pages Translate "useCallback" on Reference pages May 21, 2023
@seckie seckie changed the title Translate "useCallback" on Reference pages Translate "useCallback" hook reference May 21, 2023
@github-actions
Copy link

github-actions bot commented May 21, 2023 •
edited
Loading

Size changes

Details

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

@smikitky
Copy link
Member

smikitky commented May 22, 2023

ありがとうございます。次回以降 #452 で先に宣言していただけると助かります。

@smikitky smikitky mentioned this pull request May 22, 2023
Open
@seckie seckie force-pushed the api_reference_use_callback branch from c41e694 to de34623 Compare May 22, 2023 15:00
@smikitky
Copy link
Member

smikitky commented May 22, 2023 •
edited
Loading

Markdown パースや textlint 絡みの問題があるので、**強調** のマークアップが句点を含まないようにしていただけると助かります。そして句点の後の半角スペースを削除してください。

う。** あ ではなく う**。あ です。

@seckie seckie marked this pull request as ready for review May 22, 2023 15:16
@seckie
Copy link
Contributor Author

seckie commented May 22, 2023

@smikitky
早速のご確認とコメントありがとうございます。
0454fdd で修正しました。

ありがとうございます。次回以降 #452 で先に宣言していただけると助かります。

次回以降、先に #452 で宣言するようにします 🙇

@smikitky smikitky added 🚫 in intial review 初期レビューの真っ最中 and removed 🚫 in intial review 初期レビューの真っ最中 labels May 22, 2023
@smikitky
Copy link
Member

smikitky commented May 22, 2023

すみません、偶然今エフェクト関係の記事を翻訳していて痛感したのですが、dependency/dependencies について訳語の統一が必要だなと思っています。

https://github.com/reactjs/ja.react.dev/wiki/%E8%A8%B3%E8%AA%9E%E3%81%AE%E7%B5%B1%E4%B8%80

一旦、こちらに書いた方針で見直しをお願いできないでしょうか。

useEffect リファレンスの翻訳をしていた時はこの辺をよく考えておらず、「オブジェクトの不要な依存関係を削除する」みたいな、今見返したら何を言っているのかピンと来ない翻訳を自分がしてしまっているのですが、こういうのも「オブジェクト型の不要な依存値を削除する」と具体的に言った方が良い気がしています。（そちらは #585 と併せてこちらで修正します）

@smikitky
Copy link
Member

smikitky commented May 22, 2023

@koba04 dependencies の訳語について Wiki に書いた方針で良いでしょうか。useEffect リファレンスでも既にある程度は（半ば無意識的に）上記方針通りで訳してはいるのですが、もう少し明示的に統一したいと思います。

（この方も言っていますが、個人的には dependency injection も「依存値注入」「依存物注入」「依存オブジェクト注入」と翻訳されるべきだったと思っています…）

@seckie
Copy link
Contributor Author

seckie commented May 23, 2023

https://github.com/reactjs/ja.react.dev/wiki/%E8%A8%B3%E8%AA%9E%E3%81%AE%E7%B5%B1%E4%B8%80

一旦、こちらに書いた方針で見直しをお願いできないでしょうか。

@smikitky 上記の方針に従って修正しました。 13a06b5 ご確認お願いします。

@smikitky smikitky requested review from koba04, potato4d and smikitky May 30, 2023 02:49
@smikitky smikitky self-assigned this May 30, 2023
smikitky
smikitky requested changes May 30, 2023
View reviewed changes
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.

遅くなりましたが確認しました。明らかな誤訳みたいなのは少ないのですが、少しでも読みやすくするための suggestion の量が多めです。確認をよろしくお願いします。

なお新ドキュメントから「副作用」は「エフェクト」とすることにしました (#585)。

@seckie @smikitky
Apply suggestions from code review
315701a 
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
seckie
seckie commented May 31, 2023
View reviewed changes
Copy link
Contributor Author

@seckie seckie left a comment

Choose a reason for hiding this comment

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

@smikitky ありがとうございます。たくさんの suggestion、お手数をおかけしました。

src/content/reference/react/useCallback.md

If you're already familiar with [`useMemo`,](/reference/react/useMemo) you might find it helpful to think of `useCallback` as this:
* **`useCallback` は*関数自体*をキャッシュします**。`useMemo`とは異なり、あなたが提供する関数を呼び出しません。代わりに、あなたが提供した関数をキャッシュして、`productId` または `referrer` が変更されない限り、`handleSubmit` *自体*が変更されないようにします。これにより、`ShippingForm` を不必要に再レンダーすることなく、`handleSubmit` 関数を下位に渡すことができます。ユーザーがフォームを送信するまであなたのコードは実行されません。

Copy link
Contributor Author

@seckie seckie May 31, 2023

Choose a reason for hiding this comment

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

999a17d こちらで対応しました。

@smikitky @seckie
Apply suggestions from code review
270d8c9 
Co-authored-by: Naoki Sekiguchi <likealunatic@gmail.com>
smikitky
smikitky approved these changes Jun 1, 2023
View reviewed changes
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.

提案はそのままコミットしておきました。ありがとうございました！
2 人のレビューが必要ですのでこのままお待ち頂ければ幸いです。

koba04
koba04 approved these changes Jun 16, 2023
View reviewed changes
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 0dce3ee into reactjs:main Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@koba04 koba04 koba04 approved these changes

@smikitky smikitky smikitky approved these changes

@potato4d potato4d Awaiting requested review from potato4d

Assignees

@smikitky smikitky

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@seckie @smikitky @koba04