fix spelling variants #159
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
|
Deploy preview for ja-reactjs ready! Built with commit 26d28e1 |
uenitty
commented
Mar 3, 2019
|
|
||
| [error boundaries](/docs/error-boundaries.html) は、子コンポーネントツリーのどこかで JavaScript エラーを捕捉し、それらのエラーを記録し、クラッシュしたコンポーネントツリーの代わりにフォールバック UI を表示する React コンポーネントです。error boundaries は、その下のツリー全体のレンダー中、ライフサイクルメソッド内、およびコンストラクタ内で発生したエラーを捕捉します。 | ||
| [error boundary](/docs/error-boundaries.html) は、子コンポーネントツリーのどこかで JavaScript エラーを捕捉し、それらのエラーを記録し、クラッシュしたコンポーネントツリーの代わりにフォールバック UI を表示する React コンポーネントです。error boundary は、その下のツリー全体のレンダー中、ライフサイクルメソッド内、およびコンストラクタ内で発生したエラーを捕捉します。 |
There was a problem hiding this comment.
ここの原文は /docs/error-boundaries.html の一部と全く同じなのですが、日本語版では少し文章が異なっています。
これは揃えた方がいいのでしょうか。
あと、throwは捕捉かスローかという違いもあり、どちらに揃えるべきなのか気になっています。
uenitty
commented
Mar 3, 2019
| @@ -180,9 +180,9 @@ function MyComponent() { | |||
| } | |||
| ``` | |||
|
|
|||
| ### Error boundaries {#error-boundaries} | |||
| ### Error Boundary {#error-boundaries} | |||
There was a problem hiding this comment.
タイトル以外の本文中に出てくる "Error Boundary" は小文字に統一（文頭であっても） とのことでしたが、直前の Suspense が大文字のため一旦大文字にしてみました。
もしかしてこれって タイトル に当たりますかね...もしそうなら /docs/error-boundaries.html 側を直しにいかないと... 🙈
|
ありがとうございます。語頭の大文字小文字についてはまだたくさん不統一があり、まとめて直さないといけないのですが、とりあえず単数形に統一する部分については良いと思います。 |
smikitky
approved these changes
Mar 3, 2019
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.
I found spelling variants in translated documents and fix it.
error boundaryについて表記ゆれを発見したので修正しました（とりあえず翻訳済みドキュメントだけ）。
具体的には以下に従うようにしました。
原文での表現にかかわらず error boundary と単数表記タイトル以外の本文中に出てくる "Error Boundary" は小文字に統一（文頭であっても）