-
Notifications
You must be signed in to change notification settings - Fork 178
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 "Hooks" index page #618
Conversation
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
すみません、個別のレビューに入る前に以下の対応をお願いできないでしょうか。
- 原文と訳文で行番号を厳密に一致させる(2か所余計な改行や重複があります)
- state, ref などは文頭でも小文字にする。props などの常に複数形を使うと決めたものを除き、refs, keys などの複数形は訳文では使わない。
- 「コンテクスト」「エフェクト」などカタカナで訳すと決まっているものを英単語のままで残さない。
よろしくお願いします。
単語の統一と行数の修正、その他誤字の修正を行いました。再度確認をよろしくお願いいたします。
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
レビューを行いましたので確認をお願いします。
訳語統一関連の指摘が多くなっていますので、訳語統一ページを改めてご確認いただければ幸いです。
翻訳の統一は十分注意したつもりだったんですが見落としがかなりあったようです。申し訳ありません。 |
訳語集に載っていないものもありますし、全文目を通しているのはメンテナくらいなので気にしないで大丈夫です! https://github.com/reactjs/ja.react.dev/pull/618/files 問題なければ、上記のページから「Add suggestion to batch」機能を使ってまとめてコミットしていただけると助かります。 |
Co-authored-by: Soichiro Miki <[email protected]>
レビューの変更と、その他自分で見つけた場所を修正しました。 再度確認をお願い致します。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございました!
2 名のレビュアーが approve してからマージされますので、このままお待ちください。
There was a problem hiding this 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/index.md
を翻訳しました。Issue: #452 (comment)
各ページへのリンクについて
各ページへのリンクの概要の文章は私が翻訳しましたが、各ページのリンク先の翻訳を引用したほうが良いでしょうか?
Textlintについて
#616 での通り、
textlint
ディレクトリに入った状態でのみtextlint
が正常に動作しました。Gitへのコミット時にはtextlintを無効化してコミットしていますが、textlintでエラーや警告が出ないことは確認済みです。