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 Code-Splitting #58

Merged
merged 14 commits into from
Feb 10, 2019

Conversation

sugoikondo
Copy link
Contributor

I finished translating Code-Splitting
Please let me know if you find any mistakes.
Thanks.

target issue : #4

@sugoikondo sugoikondo changed the title Translate/code splitting Translate Code-Splitting Feb 3, 2019
@smikitky smikitky self-requested a review February 4, 2019 00:47
@koba04 koba04 requested review from koba04 and removed request for koba04 February 4, 2019 01:18
@tesseralis tesseralis mentioned this pull request Feb 5, 2019
90 tasks
@koba04
Copy link
Member

koba04 commented Feb 6, 2019

@atsumine ありがとうございます!!一点、改行を原文と合わせるようにお願いします 🙇

@smikitky
Copy link
Member

smikitky commented Feb 6, 2019

@koba04 この原文ファイルは、他と違って文の途中でも積極的に改行しているようなので、合わせるのは困難そうです…

@koba04
Copy link
Member

koba04 commented Feb 7, 2019

確かにそうですね...。しかも意味の区切りというわけでもなさそうですね。
ここに関しては改行表示に合わせつつ、後ほど原文を修正した方がよさそうですね。

@sugoikondo
Copy link
Contributor Author

sugoikondo commented Feb 7, 2019

@koba04 @smikitky コメントありがとうございます!
なるほど、確かに原文では一行が80文字未満になるように改行が入れられているみたいですね。
こちら失念しておりました…すみません!

ここに関しては改行表示に合わせつつ、後ほど原文を修正した方がよさそうですね。

承知致しました!ではなるべく原文の改行に近くなるよう、文章の改行を入れておきますね!

@koba04 koba04 added the 🚫 in intial review 初期レビューの真っ最中 label Feb 8, 2019
@koba04 koba04 self-requested a review February 8, 2019 11:25
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.

@atsumine いくつかコメントしたので確認お願いします! 🙏

content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
@@ -212,19 +186,13 @@ const MyComponent = () => (
);
```

## Route-based code splitting
## ルート単位でのコード分割
Copy link
Member

Choose a reason for hiding this comment

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

以降の文にてルートがトップレベルのルートと混同されるように感じたのでルーティングと明確にしたのですが如何でしょうか? (imo)

Suggested change
## ルート単位でのコード分割
## ルーティング単位でのコード分割

@koba04 koba04 removed the 🚫 in intial review 初期レビューの真っ最中 label Feb 8, 2019
@sugoikondo
Copy link
Contributor Author

ご指摘ありがとうございます!こんなに丁寧にレビューして頂いて感無量です…!

コンフリクトの解消とともに、修正してまいりますね!

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.

微修正が多いのですがこちらをご確認いただけますでしょうか。

Bundling is the process of following imported files and merging them into a
single file: a "bundle". This bundle can then be included on a webpage to load
an entire app at once.
多くの React アプリケーションは、[Webpack](https://webpack.js.org/)や [Browserify](http://browserify.org/) などのツールを使ってファイルを「バンドル」しています。バンドルはインポートされたファイルをたどって、それらを1つのファイルにまとめるプロセスです。このバンドルされたファイルを Web ページ内に置くことによって、アプリ全体を一度に読み込むことができます。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
多くの React アプリケーションは、[Webpack](https://webpack.js.org/)[Browserify](http://browserify.org/) などのツールを使ってファイルを「バンドル」しています。バンドルはインポートされたファイルをたどって、それらを1つのファイルにまとめるプロセスです。このバンドルされたファイルを Web ページ内に置くことによって、アプリ全体を一度に読み込むことができます。
多くの React アプリケーションは、[Webpack](https://webpack.js.org/)[Browserify](http://browserify.org/) などのツールを使ってファイルを「バンドル」しています。バンドルはインポートされたファイルをたどって、それらを 1 つのファイルにまとめるプロセスです。このバンドルされたファイルを Web ページ内に置くことによって、アプリ全体を一度に読み込むことができます。

content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
content/docs/code-splitting.md Outdated Show resolved Hide resolved
@smikitky
Copy link
Member

smikitky commented Feb 9, 2019

確認しました。

もし Webpack を自分でセットアップしていた場合には、Webpack のコード分割に関するガイドを読むと良いでしょう。きっとあなたの Webpack の設定はおよそこのようになっているはずでしょうから。

こちらだけ対応をお願いします。このままだと、リンクの gist に code splitting 導入前のダメな例が載っているかのような文になってしまっています。リンク先の gist は code splitting が正しく動作するための最小限の例です。「(code splittingを導入すると、)大雑把にはこんな感じの設定になるでしょう」ということが言いたいのだと思います。

@sugoikondo
Copy link
Contributor Author

なんと!すみません、完全に原文の意味を取り違っておりました。。
すぐに修正します!

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.

@atsumine いくつか間違った指摘をしてすみませんでした🙏
対応ありがとうございます✨

@sugoikondo
Copy link
Contributor Author

@koba04 いえいえ!もとはといえば、私の誤訳や間際らしい文章がすべての原因ですので、落ち度は私にあります。申し訳ありませんでした。:bow:

お二人方には本当に丁寧にレビューをしていただき、感無量です!
もしまだ修正点やおかしな文章がありましたら、またその他お手伝いできることがあれば何でも致しますので、ぜひお気軽にお申し付けください!

@smikitky smikitky merged commit 321eb1a into reactjs:master Feb 10, 2019
@smikitky
Copy link
Member

マージしました、ありがとうございました!

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