Translate SyntheticEvent #43

summertree128 · 2019-02-02T06:13:39Z

This PR is part of #4.
Added translation to Add SyntheticEvents page.

Please review.

koba04

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

koba04 · 2019-02-04T03:06:35Z

content/docs/reference-events.md


-Your event handlers will be passed instances of `SyntheticEvent`, a cross-browser wrapper around the browser's native event. It has the same interface as the browser's native event, including `stopPropagation()` and `preventDefault()`, except the events work identically across all browsers.
+あなたの書くイベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちます。この 2つ以外はすべてのブラウザで同じように機能します。


通常のイベントオブジェクトと違い、ブラウザ互換性があるということが言いたいのだと思うので修正しました。

Suggested change

あなたの書くイベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちます。この 2つ以外はすべてのブラウザで同じように機能します。

イベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ち、すべてのブラウザで同じように動作します。

ここは私も悩んだのですが、"except the events"は「stopPropagationとpreventDefault以外」という意味ではないでしょうか？つまり、この2つのイベントは全ブラウザで同じようには動かない（古いIEなど）けどそれ以外は全ブラウザで同じように動くという意味なのではないかと解釈しました。こちらどう思われますか?

koba04 さんの指摘が正しいと思います。比較的直訳的に行くと「SyntheticEventのインターフェースは元と変わっていないが、ブラウザ間で同様に動作するという点においては異なっています」です。

「この2つのイベント以外では」であれば "except (for) these," のような英文になるはずです。

少しだけ except のニュアンスを出すようにしてみましたが如何でしょう

Suggested change

あなたの書くイベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちます。この 2つ以外はすべてのブラウザで同じように機能します。

あなたの書くイベントハンドラには、`SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちつつ、ブラウザ間で同じ挙動をするようになっています。

koba04 · 2019-02-04T03:15:15Z

content/docs/reference-events.md

 >
-> As of v0.14, returning `false` from an event handler will no longer stop event propagation. Instead, `e.stopPropagation()` or `e.preventDefault()` should be triggered manually, as appropriate.
+> v0.14以降、イベントハンドラから false を返してもイベントの伝播が止まることはなくなりました。代わりに、`e.stopPropagation()` または `e.preventDefault()` を手動で呼び出す必要があります。


Suggested change

> v0.14以降、イベントハンドラから false を返してもイベントの伝播が止まることはなくなりました。代わりに、`e.stopPropagation()` または `e.preventDefault()` を手動で呼び出す必要があります。

> v0.14以降、イベントハンドラから `false` を返してもイベントの伝播は止まりません。代わりに、`e.stopPropagation()` または `e.preventDefault()` を手動で呼び出す必要があります。

koba04 · 2019-02-04T03:20:16Z

content/docs/reference-events.md

-As such, you cannot access the event in an asynchronous way.
+### イベントのプール
+
+`SyntheticEvent` はプールされます。つまり、`SyntheticEvent` オブジェクトは再利用され、すべてのプロパティはイベントコールバックが呼び出された後に無効化されます。これはパフォーマンス上の理由からです。そのため、非同期的な方法でイベントにアクセスすることはできません。


挙動としてもわかりにくい部分なので、説明を足してみました。

Suggested change

`SyntheticEvent` はプールされます。つまり、`SyntheticEvent` オブジェクトは再利用され、すべてのプロパティはイベントコールバックが呼び出された後に無効化されます。これはパフォーマンス上の理由からです。そのため、非同期的な方法でイベントにアクセスすることはできません。

`SyntheticEvent` はプールされます。つまり、`SyntheticEvent` オブジェクトは再利用され、すべてのプロパティはイベントコールバックが呼び出された後に `null` で初期化されます。これはパフォーマンス上の理由からです。そのため、非同期処理の中でイベントオブジェクトにアクセスすることはできません。

また、改行が原文と一致していないようです。

koba04 · 2019-02-04T03:29:16Z

content/docs/reference-events.md


 ```javascript
 function onClick(event) {
-  console.log(event); // => nullified object.
+  console.log(event); // => 無効なオブジェクト


この時点では、無効化されていないと思うので、言い方を変えてみました。

Suggested change

console.log(event); // => 無効なオブジェクト

console.log(event); // => null で初期化されるオブジェクト

確かに「このあとで無効化される」ということみたいですね、ややこしい…

koba04 · 2019-02-04T03:32:39Z

content/docs/reference-events.md

@@ -52,70 +51,70 @@ function onClick(event) {
    console.log(eventType); // => "click"
  }, 0);

-  // Won't work. this.state.clickEvent will only contain null values.
+  // これは動作しません。this.state.clickEvent はnull値のみを保持します。


Suggested change

// これは動作しません。this.state.clickEvent はnull値のみを保持します。

// これは動作しません。this.state.clickEvent は null 値のみを持つオブジェクトとなります。

koba04 · 2019-02-04T03:45:54Z

content/docs/reference-events.md

- [Animation Events](#animation-events)
- [Transition Events](#transition-events)
- [Other Events](#other-events)
+> 非同期処理でイベントのプロパティにアクセスしたい場合は、`event.persist()` をイベント内で呼び出す必要があります。これにより、プールから合成イベントが削除され、イベントへの参照をコードで保持できるようになります。


Suggested change

> 非同期処理でイベントのプロパティにアクセスしたい場合は、`event.persist()` をイベント内で呼び出す必要があります。これにより、プールから合成イベントが削除され、イベントへの参照をコードで保持できるようになります。

> 非同期処理の中でイベントのプロパティにアクセスしたい場合は、`event.persist()` をイベント内で呼び出す必要があります。これにより、合成イベントがイベントプールの対象から除外され、イベントへの参照をコードで保持できるようになります。

koba04 · 2019-02-04T03:47:24Z

content/docs/reference-events.md

+
+## サポートするイベント
+
+React はイベントを正規化して、異なるブラウザ間で一貫したプロパティが保持されるようにしています。


Suggested change

React はイベントを正規化して、異なるブラウザ間で一貫したプロパティが保持されるようにしています。

React は異なるブラウザ間でも一貫したプロパティを持つようにイベントを正規化します。

koba04 · 2019-02-04T03:54:31Z

content/docs/reference-events.md


-Pointer events are not yet supported in every browser (at the time of writing this article, supported browsers include: Chrome, Firefox, Edge, and Internet Explorer). React deliberately does not polyfill support for other browsers because a standard-conform polyfill would significantly increase the bundle size of `react-dom`.
+すべてのブラウザでポインタイベントがサポートされているわけではありません（この記事の執筆時点でサポートされているブラウザは、Chrome、Firefox、Edge、および Internet Explorer です）。標準に準拠したポリフィルは `react-dom` のバンドルサイズを大幅に増加させるため、Reactは意図的に他のブラウザのポリフィルをサポートしません。


Suggested change

すべてのブラウザでポインタイベントがサポートされているわけではありません（この記事の執筆時点でサポートされているブラウザは、Chrome、Firefox、Edge、および Internet Explorer です）。標準に準拠したポリフィルは `react-dom` のバンドルサイズを大幅に増加させるため、Reactは意図的に他のブラウザのポリフィルをサポートしません。

すべてのブラウザでポインタイベントがサポートされているわけではありません（この記事の執筆時点でサポートされているブラウザは、Chrome、Firefox、Edge、および Internet Explorer です）。標準に準拠したポリフィルは `react-dom` のバンドルサイズを大幅に増加させるため、React は意図的にその他ブラウザのためのポリフィルを提供しません。

smikitky

レビューしました、微修正のみです。
よろしくお願いします。

smikitky · 2019-02-05T05:38:22Z

content/docs/reference-events.md


-Your event handlers will be passed instances of `SyntheticEvent`, a cross-browser wrapper around the browser's native event. It has the same interface as the browser's native event, including `stopPropagation()` and `preventDefault()`, except the events work identically across all browsers.
+あなたの書くイベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちます。この 2つ以外はすべてのブラウザで同じように機能します。


koba04 さんの指摘が正しいと思います。比較的直訳的に行くと「SyntheticEventのインターフェースは元と変わっていないが、ブラウザ間で同様に動作するという点においては異なっています」です。

「この2つのイベント以外では」であれば "except (for) these," のような英文になるはずです。

少しだけ except のニュアンスを出すようにしてみましたが如何でしょう

Suggested change

あなたの書くイベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちます。この 2つ以外はすべてのブラウザで同じように機能します。

あなたの書くイベントハンドラには、`SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちつつ、ブラウザ間で同じ挙動をするようになっています。

smikitky · 2019-02-05T05:41:49Z

content/docs/reference-events.md


 ```javascript
 function onClick(event) {
-  console.log(event); // => nullified object.
+  console.log(event); // => 無効なオブジェクト


確かに「このあとで無効化される」ということみたいですね、ややこしい…

content/docs/reference-events.md

summertree128 · 2019-02-05T07:11:21Z

@koba04 @smikitky
Thank you for your review! Changed all of them.
Especially line 13 was hard to understand for me, thank you for good advices.

smikitky · 2019-02-05T07:27:22Z

私の方ではOKです！

koba04

@summertree128 原文にない改行が2つ含めれているのでそこだけ対応お願いします 🙇
その他はOKだと思います！

koba04 · 2019-02-05T09:28:28Z

content/docs/reference-events.md

+`SyntheticEvent` はプールされます。つまり、`SyntheticEvent` オブジェクトは再利用され、すべてのプロパティはイベントコールバックが呼び出された後に `null` で初期化されます。
+これはパフォーマンス上の理由からです。
+そのため、非同期処理の中でイベントオブジェクトにアクセスすることはできません。
+


ここで原文と比較すると2行ずれているので、行数を合わせていただけると 🙏
### イベントのプールの上とそのため、非同期処理の中でイベントオブジェクトにアクセスすることはできません。の下にそれぞれに不要な改行があります。

すみません、気づいてませんでした。細かいところありがとうございます 🙏
修正しました。

koba04 · 2019-02-05T14:00:47Z

@summertree128 対応ありがとうございました！mergeしました 🚀🚀🚀

summertree128 added 2 commits February 2, 2019 15:10

WIP: translate synthetic events

0e508b9

translate synthetic events

6c2a624

summertree128 changed the title ~~WIP: translate synthetic events~~ Translate SyntheticEvent Feb 3, 2019

remove unnecessary space

44d821c

smikitky self-requested a review February 3, 2019 11:36

koba04 self-requested a review February 4, 2019 01:15

koba04 requested changes Feb 4, 2019

View reviewed changes

smikitky requested changes Feb 5, 2019

View reviewed changes

summertree128 force-pushed the translate/synthetic-events branch from 628916a to 2c684c1 Compare February 5, 2019 06:59

fix some part according to review

fb31de6

summertree128 force-pushed the translate/synthetic-events branch from 2c684c1 to fb31de6 Compare February 5, 2019 07:02

smikitky approved these changes Feb 5, 2019

View reviewed changes

koba04 requested changes Feb 5, 2019

View reviewed changes

tesseralis mentioned this pull request Feb 5, 2019

Japanese Translation Progress #4

Closed

90 tasks

remove unnecessary line break

8180825

koba04 approved these changes Feb 5, 2019

View reviewed changes

koba04 merged commit 23e88ff into reactjs:master Feb 5, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Translate SyntheticEvent #43

Translate SyntheticEvent #43

summertree128 commented Feb 2, 2019 •

edited

koba04 left a comment

koba04 Feb 4, 2019

summertree128 Feb 4, 2019 •

edited

smikitky Feb 5, 2019 •

edited

koba04 Feb 4, 2019

koba04 Feb 4, 2019

koba04 Feb 4, 2019

smikitky Feb 5, 2019

koba04 Feb 4, 2019

koba04 Feb 4, 2019

koba04 Feb 4, 2019

koba04 Feb 4, 2019 •

edited by smikitky

smikitky left a comment

smikitky Feb 5, 2019 •

edited

smikitky Feb 5, 2019

summertree128 commented Feb 5, 2019

smikitky commented Feb 5, 2019

koba04 left a comment

koba04 Feb 5, 2019

summertree128 Feb 5, 2019 •

edited

koba04 commented Feb 5, 2019


		Your event handlers will be passed instances of `SyntheticEvent`, a cross-browser wrapper around the browser's native event. It has the same interface as the browser's native event, including `stopPropagation()` and `preventDefault()`, except the events work identically across all browsers.
		あなたの書くイベントハンドラには、 `SyntheticEvent` のインスタンスが渡されます。これはブラウザのネイティブイベントに対するクロスブラウザ版のラッパです。`stopPropagation()` と `preventDefault()` を含む、ブラウザのネイティブイベントと同じインターフェイスを持ちます。この 2つ以外はすべてのブラウザで同じように機能します。

	> v0.14以降、イベントハンドラから false を返してもイベントの伝播が止まることはなくなりました。代わりに、`e.stopPropagation()` または `e.preventDefault()` を手動で呼び出す必要があります。
	> v0.14以降、イベントハンドラから `false` を返してもイベントの伝播は止まりません。代わりに、`e.stopPropagation()` または `e.preventDefault()` を手動で呼び出す必要があります。

	console.log(event); // => 無効なオブジェクト
	console.log(event); // => null で初期化されるオブジェクト

	// これは動作しません。this.state.clickEvent はnull値のみを保持します。
	// これは動作しません。this.state.clickEvent は null 値のみを持つオブジェクトとなります。

	> 非同期処理でイベントのプロパティにアクセスしたい場合は、`event.persist()` をイベント内で呼び出す必要があります。これにより、プールから合成イベントが削除され、イベントへの参照をコードで保持できるようになります。
	> 非同期処理の中でイベントのプロパティにアクセスしたい場合は、`event.persist()` をイベント内で呼び出す必要があります。これにより、合成イベントがイベントプールの対象から除外され、イベントへの参照をコードで保持できるようになります。


		## サポートするイベント

		React はイベントを正規化して、異なるブラウザ間で一貫したプロパティが保持されるようにしています。

	React はイベントを正規化して、異なるブラウザ間で一貫したプロパティが保持されるようにしています。
	React は異なるブラウザ間でも一貫したプロパティを持つようにイベントを正規化します。


		Pointer events are not yet supported in every browser (at the time of writing this article, supported browsers include: Chrome, Firefox, Edge, and Internet Explorer). React deliberately does not polyfill support for other browsers because a standard-conform polyfill would significantly increase the bundle size of `react-dom`.
		すべてのブラウザでポインタイベントがサポートされているわけではありません（この記事の執筆時点でサポートされているブラウザは、Chrome、Firefox、Edge、および Internet Explorer です）。標準に準拠したポリフィルは `react-dom` のバンドルサイズを大幅に増加させるため、Reactは意図的に他のブラウザのポリフィルをサポートしません。

Translate SyntheticEvent #43

Translate SyntheticEvent #43

Conversation

summertree128 commented Feb 2, 2019 • edited

koba04 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

summertree128 Feb 4, 2019 • edited

Choose a reason for hiding this comment

smikitky Feb 5, 2019 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

koba04 Feb 4, 2019 • edited by smikitky

Choose a reason for hiding this comment

smikitky left a comment

Choose a reason for hiding this comment

smikitky Feb 5, 2019 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

summertree128 commented Feb 5, 2019

smikitky commented Feb 5, 2019

koba04 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

summertree128 Feb 5, 2019 • edited

Choose a reason for hiding this comment

koba04 commented Feb 5, 2019

summertree128 commented Feb 2, 2019 •

edited

summertree128 Feb 4, 2019 •

edited

smikitky Feb 5, 2019 •

edited

koba04 Feb 4, 2019 •

edited by smikitky

smikitky Feb 5, 2019 •

edited

summertree128 Feb 5, 2019 •

edited