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 "Hooks" index page #618

Merged
merged 4 commits into from
Jul 20, 2023
Merged

Translate "Hooks" index page #618

merged 4 commits into from
Jul 20, 2023

Conversation

Hayao0819
Copy link
Contributor

src/content/reference/react/index.mdを翻訳しました。

Issue: #452 (comment)

各ページへのリンクについて

各ページへのリンクの概要の文章は私が翻訳しましたが、各ページのリンク先の翻訳を引用したほうが良いでしょうか?

Textlintについて

#616 での通り、textlintディレクトリに入った状態でのみtextlintが正常に動作しました。Gitへのコミット時にはtextlintを無効化してコミットしていますが、textlintでエラーや警告が出ないことは確認済みです。

@github-actions
Copy link

github-actions bot commented Jul 15, 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.

すみません、個別のレビューに入る前に以下の対応をお願いできないでしょうか。

  • 原文と訳文で行番号を厳密に一致させる(2か所余計な改行や重複があります)
  • state, ref などは文頭でも小文字にする。props などの常に複数形を使うと決めたものを除き、refs, keys などの複数形は訳文では使わない。
  • 「コンテクスト」「エフェクト」などカタカナで訳すと決まっているものを英単語のままで残さない。

よろしくお願いします。

@Hayao0819
Copy link
Contributor Author

Hayao0819 commented Jul 18, 2023

単語の統一と行数の修正、その他誤字の修正を行いました。再度確認をよろしくお願いいたします。

React DevToolsのみは、固有名詞として複数形のままにしました。

@smikitky smikitky requested review from koba04, potato4d and smikitky July 18, 2023 04:40
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/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
src/content/reference/react/index.md Outdated Show resolved Hide resolved
@Hayao0819
Copy link
Contributor Author

翻訳の統一は十分注意したつもりだったんですが見落としがかなりあったようです。申し訳ありません。

@smikitky
Copy link
Member

訳語集に載っていないものもありますし、全文目を通しているのはメンテナくらいなので気にしないで大丈夫です!

https://github.com/reactjs/ja.react.dev/pull/618/files

問題なければ、上記のページから「Add suggestion to batch」機能を使ってまとめてコミットしていただけると助かります。

@Hayao0819
Copy link
Contributor Author

Hayao0819 commented Jul 19, 2023

レビューの変更と、その他自分で見つけた場所を修正しました。

再度確認をお願い致します。

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 名のレビュアーが approve してからマージされますので、このままお待ちください。

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 3d48615 into reactjs:main Jul 20, 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