Handling Events#9
Merged
vldmrkl merged 4 commits intoreactjs:masterfrom Feb 14, 2019
Merged
Conversation
westanvv
reviewed
Feb 13, 2019
| Тут `e` - це синтетична подія. React визначає ці синтетичні події відповідно до [специфікації W3C](https://www.w3.org/TR/DOM-Level-3-Events/), тому вам не потрібно турбуватися про сумісніть між браузерами. Перегляньте довідник по [`SyntheticEvent`](/docs/events.html), щоб дізнатися більше. | ||
|
|
||
| When using React you should generally not need to call `addEventListener` to add listeners to a DOM element after it is created. Instead, just provide a listener when the element is initially rendered. | ||
| Зазвичай, коли ви використовуєте React, вам не потрібно викликати `addEventListener`, щоб додати обробник до DOM-елементу після його створення. Натомість, просто вкажіть обробник,коли елемент вперше відрендерився. |
Contributor
There was a problem hiding this comment.
Missed space обробник,коли
vldmrkl
requested changes
Feb 13, 2019
Member
vldmrkl
left a comment
There was a problem hiding this comment.
Чудовий початок! Я знайшов небагато дрібних помилок, але вцілому дуже гарно 👍
| ``` | ||
|
|
||
| In React, this could instead be: | ||
| В React це може виглядати так: |
| class LoggingButton extends React.Component { | ||
| handleClick() { | ||
| console.log('this is:', this); | ||
| console.log('this це:', this); |
Contributor
Author
There was a problem hiding this comment.
Тут під this мається на увазі ключове слово this і те чому this в контексті метода дорівнює.
| ``` | ||
|
|
||
| The problem with this syntax is that a different callback is created each time the `LoggingButton` renders. In most cases, this is fine. However, if this callback is passed as a prop to lower components, those components might do an extra re-rendering. We generally recommend binding in the constructor or using the class fields syntax, to avoid this sort of performance problem. | ||
| Проблема цього синтаксису полягає в тому, що при кожному рендері `LoggingButton` створюється щоразу нова функція зворотнього виклику. В більшості випадків це не створює додаткових проблем. Але, якщо ця функція зворотнього виклику передається в якості проп в компоненти нижче - вони можуть здійснити додатковий ререндеринг. Як правило, ми рекомендуємо зв'язувати в конструкторі або використувати синтаксис полей класу, щоб уникнути подібних проблем з продуктивністю. |
Member
There was a problem hiding this comment.
У більшості випадків замість В більшості випадків
Member
There was a problem hiding this comment.
Замінити передається в якості проп на передається в якості пропу
Contributor
There was a problem hiding this comment.
Можливо краще якості пропса. Хоч і не множина, але не має неузгодженості, коли props перекладаємо пропсів(пропси) замість пропів(пропи )
Member
There was a problem hiding this comment.
@westanvv згідно нашого TRANSLATE.md, prop перекладаємо як проп, а props – як пропси.
| Проблема цього синтаксису полягає в тому, що при кожному рендері `LoggingButton` створюється щоразу нова функція зворотнього виклику. В більшості випадків це не створює додаткових проблем. Але, якщо ця функція зворотнього виклику передається в якості проп в компоненти нижче - вони можуть здійснити додатковий ререндеринг. Як правило, ми рекомендуємо зв'язувати в конструкторі або використувати синтаксис полей класу, щоб уникнути подібних проблем з продуктивністю. | ||
|
|
||
| ## Passing Arguments to Event Handlers {#passing-arguments-to-event-handlers} | ||
| ## Передача аргументів у обробники подій {#passing-arguments-to-event-handlers} |
Contributor
Author
There was a problem hiding this comment.
Можна й так, параметр передається в функцію, але певно до обробника. Є пропозиція означити це в TRANSLATION.md.
92 tasks
westanvv
approved these changes
Feb 13, 2019
| ``` | ||
|
|
||
| The problem with this syntax is that a different callback is created each time the `LoggingButton` renders. In most cases, this is fine. However, if this callback is passed as a prop to lower components, those components might do an extra re-rendering. We generally recommend binding in the constructor or using the class fields syntax, to avoid this sort of performance problem. | ||
| Проблема цього синтаксису полягає в тому, що при кожному рендері `LoggingButton` створюється щоразу нова функція зворотнього виклику. В більшості випадків це не створює додаткових проблем. Але, якщо ця функція зворотнього виклику передається в якості проп в компоненти нижче - вони можуть здійснити додатковий ререндеринг. Як правило, ми рекомендуємо зв'язувати в конструкторі або використувати синтаксис полей класу, щоб уникнути подібних проблем з продуктивністю. |
Contributor
There was a problem hiding this comment.
Можливо краще якості пропса. Хоч і не множина, але не має неузгодженості, коли props перекладаємо пропсів(пропси) замість пропів(пропи )
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Added uk translation for handling events.