-
Notifications
You must be signed in to change notification settings - Fork 176
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
Optimizing performance #122
Conversation
257bc66
to
401ec03
Compare
d90fea0
to
fb79320
Compare
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.
遅くなりましたが、全体を確認しました。
- 数が多かったので個別に指摘していませんが、「レンダリング」は「レンダー」で統一でお願いします(名詞として使用される場合も動詞として使用される場合も同じ)。
- 原文にないところで改行が含まれていたり、逆に原文に存在する空行を消してしまっているところがたくさんあり、GitHubの差分で見ても左右で文章が一致していないところが多くあります。今後原文の翻訳が更新されたときの自動マージに悪影響があるため、改行位置を統一するようお願いします。
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Deploy preview for ja-reactjs ready! Built with commit ddf7100 |
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Deploy preview for ja-reactjs ready! Built with commit 22a0825 |
Deploy preview for ja-reactjs ready! Built with commit 1faf91f |
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
…reactjs.org into optimizing-performance
レビューいただきありがとうございます。修正しました。改行については今後も英文版とdiff常にとるように留意します。よろしくおねがいします。 |
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.
@uehaj レイアウトと、単語の言い換えが中心ですが、コメントしたので確認お願いします 🙏
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
Co-Authored-By: uehaj <59012+uehaj@users.noreply.github.com>
ここまで確認しました。4箇所不必要な改行(ないし
|
よさそうです、ありがとうございました! |
@uehaj mergeしました! 🚀 ありがとうございます!! 🎉🎉🎉 |
ありがとうございました! もう他には翻訳未着手はないですね。(ほぼ)おめでとうございます! |
This PR is part of Issue #4.
レビューを宜しくお願いします。