Sync with reactjs.org @ 68e4efcf #416
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Ethan Yang <yuxaing.yang@amh-group.com>
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
Co-authored-by: Nancy Heiss <noreply>
(`setup` is the noun form and `set up` is the verb form.)
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
…gned (#3701) in CodePen it is 'prevState' and in the tutorial, it is 'state', which creates confusion.
|
✔️ Deploy Preview for ja-reactjs ready! 🔨 Explore the source changes: 7d0a973 🔍 Inspect the deploy log: https://app.netlify.com/sites/ja-reactjs/deploys/60c7ff89855ebd000747f10c 😎 Browse the preview: https://deploy-preview-416--ja-reactjs.netlify.app |
smikitky
approved these changes
Jun 13, 2021
smikitky
reviewed
Jun 13, 2021
Comment on lines
178
to
179
| > React 17 以前は、標準の [User Timing API](https://developer.mozilla.org/en-US/docs/Web/API/User_Timing_API) を用いて Chrome のパフォーマンスタブでコンポーネントのプロファイリングを行います。 | ||
| > これについての概要は [Ben Schwarz によるこの記事](https://calibreapp.com/blog/react-performance-profiling-optimization)を参照してください。 |
There was a problem hiding this comment.
ここについては実際にどんな変更が起きたのか理解せず訳したので間違っているかもしれません。ご存知でしたらご確認いただけますでしょうか
koba04
reviewed
Jun 14, 2021
| @@ -21,6 +21,10 @@ next: concurrent-mode-reference.html | |||
| > | |||
| > このドキュメントは興味のある読者やアーリーアダプター向けのものです。**React が初めての方はこれらの機能を気にしないで構いません** -- 今すぐに学ぶ必要はありません。 | |||
|
|
|||
| >Caution: | |||
| > | |||
| >All references below to "blocking mode" and `createBlockingRoot` are outdated and should be ignored. | |||
There was a problem hiding this comment.
コンフリクトが出ないと見逃しがちですね…すみません
（でも最新だとそもそも Concurrent Mode 自体ナビゲーションから消えてますね…）
| @@ -199,6 +173,11 @@ React DevTools をまだインストールしていない場合は、以下で | |||
| > 本番ビルド版 `react-dom` のプロファイリング可能なバンドルとして `react-dom/profiling` が利用可能です。 | |||
| > このバンドルの使い方の詳細については、[fb.me/react-profiling](https://fb.me/react-profiling) を参照してください。 | |||
|
|
|||
| > 捕捉 | |||
| > | |||
| > React 17 以前は、標準の [User Timing API](https://developer.mozilla.org/en-US/docs/Web/API/User_Timing_API) を用いて Chrome のパフォーマンスタブでコンポーネントのプロファイリングを行います。 | |||
There was a problem hiding this comment.
Suggested change
| > React 17 以前は、標準の [User Timing API](https://developer.mozilla.org/en-US/docs/Web/API/User_Timing_API) を用いて Chrome のパフォーマンスタブでコンポーネントのプロファイリングを行います。 | |
| > React 17 より前は、標準の [User Timing API](https://developer.mozilla.org/en-US/docs/Web/API/User_Timing_API) を用いて Chrome のパフォーマンスタブでコンポーネントのプロファイリングを行います。 |
こちらの PR で削除された機能で、Dev mode の時に User Timing API でコンポーネントのレンダリングにかかった時間を表示していた機能で、これによりブラウザの Performance Panel 上でどのコンポーネントにどれくらいかかったのかがはわかるようになっていました。なので翻訳としては間違ってないと思います。
ただ v17 で削除されたので 17 以前ではなく 16 以前となるのでその点だと suggest しました。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR was automatically generated.
Merge changes from reactjs.org at 68e4efc
The following files have conflicts and may need new translations:
Please fix the conflicts by pushing new commits to this pull request, either by editing the files directly on GitHub or by checking out this branch.
DO NOT SQUASH MERGE THIS PULL REQUEST!
Doing so will "erase" the commits from master and cause them to show up as conflicts the next time we merge.