Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Translation of Handling Events #20

Merged
merged 5 commits into from
Jul 27, 2019
Merged

Translation of Handling Events #20

merged 5 commits into from
Jul 27, 2019

Conversation

seraco
Copy link
Contributor

@seraco seraco commented Jul 21, 2019

No description provided.

@seraco seraco requested a review from pepjo July 21, 2019 10:21
@seraco seraco changed the title Translation of **Handling Events** Translation of Handling Events Jul 21, 2019
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved

You have to be careful about the meaning of `this` in JSX callbacks. In JavaScript, class methods are not [bound](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_objects/Function/bind) by default. If you forget to bind `this.handleClick` and pass it to `onClick`, `this` will be `undefined` when the function is actually called.
Has d'anar amb compte amb el significat de `this` a callbacks de JSX. A JavaScript,
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you remove the unneeded line break?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, thanks.

content/docs/handling-events.md Outdated Show resolved Hide resolved
content/docs/handling-events.md Outdated Show resolved Hide resolved
seraco and others added 4 commits July 23, 2019 19:36
@seraco
Copy link
Contributor Author

seraco commented Jul 23, 2019

Thanks @ragafus for all you reviews 😋.

@seraco seraco removed the request for review from pepjo July 27, 2019 15:28
@seraco seraco merged commit d70250f into master Jul 27, 2019
@tesseralis tesseralis mentioned this pull request Jul 27, 2019
88 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants