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 Warnings pages #50

Merged
merged 17 commits into from Feb 18, 2019
Merged

Translate Warnings pages #50

merged 17 commits into from Feb 18, 2019

Conversation

uehaj
Copy link
Contributor

@uehaj uehaj commented Feb 3, 2019

This PR is part of Issue #4.
レビューお願いします。

一連の警告ページについては、タイトルは意図的に未訳にしてます。理由は、警告メッセージを体言止めとしてうまく翻訳できなかったためと、警告種別を識別するためのマーカーとしてタイトルが使われる意義があると感じたため。

例: 「Don't Call PropTypes Warning」 →「PropTypesを呼んではならない警告」にはせずに英文のまま。

そのかわり、本文の冒頭1文に、端的に意味がわかる一文を置くようにした。

また、懸念としては、これらの警告ページ群には本文からの明示的なリンクは存在しないので、検索クローラが到達できない可能性がある。また、Reactランタイムが生成する警告メッセージ中にはURLには英文版へのリンクしかふくまれない。そのために、利用者がこれらの警告ページの日本語版情報に到達することができないかもしれない。この問題を解決するには、(1)公式ドキュメントに英日などの言語切り替えのUIを付与する(2)ブラウザが送出するLangを見ての切り替え(3)警告ページに対するリンクが目次や本文から貼られる、な�どの対策のいずれかが必要と思われる。
ご意見求むです。

@smikitky smikitky self-requested a review February 3, 2019 03:50
@smikitky
Copy link
Member

smikitky commented Feb 3, 2019

とりあえずまだ textlint がこけているようですのでご確認お願いします。

@tesseralis tesseralis mentioned this pull request Feb 3, 2019
90 tasks
@uehaj
Copy link
Contributor Author

uehaj commented Feb 3, 2019

とりあえずまだ textlint がこけているようですのでご確認お願いします。

ご指摘ありがとございます。textlint通るように修正してみました。

@smikitky smikitky changed the title WIP: Translation Warnings Translation Warnings Feb 4, 2019
@smikitky smikitky changed the title Translation Warnings Translate Warnings pages Feb 4, 2019
@koba04 koba04 self-requested a review February 4, 2019 01:17
Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

@uehaj
翻訳ありがとうございます!
細かな点以外でざっとですが、レビューしました!

content/warnings/dont-call-proptypes.md Outdated Show resolved Hide resolved
content/warnings/dont-call-proptypes.md Show resolved Hide resolved
content/warnings/dont-call-proptypes.md Show resolved Hide resolved
content/warnings/dont-call-proptypes.md Outdated Show resolved Hide resolved
content/warnings/legacy-factories.md Outdated Show resolved Hide resolved
content/warnings/refs-must-have-owner.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
@koba04 koba04 removed their request for review February 4, 2019 09:41
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
commented at review

Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
commented at review

Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
@uehaj
Copy link
Contributor Author

uehaj commented Feb 6, 2019

現時点(2/6 9:00)まで頂いたコメントについて、conversationが残っているもの以外はすべて反映済みです。

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.

@uehaj 細かいスタイル系の修正がほとんどですが確認お願いします 🙇

content/warnings/dont-call-proptypes.md Outdated Show resolved Hide resolved
content/warnings/dont-call-proptypes.md Outdated Show resolved Hide resolved
content/warnings/invalid-aria-prop.md Outdated Show resolved Hide resolved
content/warnings/invalid-aria-prop.md Show resolved Hide resolved
content/warnings/refs-must-have-owner.md Outdated Show resolved Hide resolved
content/warnings/refs-must-have-owner.md Show resolved Hide resolved
content/warnings/special-props.md Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
@smikitky
Copy link
Member

@uehaj もう残っているのは翻訳ではなくコンフリクト解決などの作業だけなので、もしよろしければこちらで引き取って必要な作業を終えてしまいますが、どうしましょうか?

Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
@uehaj
Copy link
Contributor Author

uehaj commented Feb 14, 2019

@smikitky 残っているのはコンフリクト解決のみになりました。マージをお願いできればと思います。

@smikitky smikitky merged commit 7ad7ffe into reactjs:master Feb 18, 2019
@smikitky
Copy link
Member

smikitky commented Feb 18, 2019

翻訳自体は(ごく小さな訳語統一などを除き)問題ないのでマージしました。スペースやlintやアンカー文字列など小さな問題が残っていますがこちらで対応します。ありがとうございました!

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.

None yet

4 participants