Skip to content

Translate Handling Events #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Mar 7, 2019

Conversation

ntban15
Copy link
Contributor

@ntban15 ntban15 commented Feb 24, 2019

  • Translate Handling Events page

@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for vi-reactjs-org ready!

Built with commit c695dd5

https://deploy-preview-17--vi-reactjs-org.netlify.com

@nampdn nampdn requested a review from ltmylinh February 25, 2019 07:35
@nampdn
Copy link
Collaborator

nampdn commented Feb 26, 2019

@ntban15 Please resolve conflict

@ntban15
Copy link
Contributor Author

ntban15 commented Feb 26, 2019

@nampdn Done

@nampdn nampdn self-requested a review February 26, 2019 15:11
Copy link
Collaborator

@nampdn nampdn left a comment

Choose a reason for hiding this comment

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

Nhìn chung rất chuẩn. Có vài chỗ dùng chữ "cảnh báo" có lẽ hơi máy móc và nặng từ. Mình có update term bên glossary từ này nên dịch là "lưu ý" sẽ thân thuộc hơn.

@nampdn
Copy link
Collaborator

nampdn commented Feb 26, 2019

Chị @ltmylinh ơi, chị review giúp thêm lần nữa cho PR này nhé.

+ Update 'warning'
+ Update MDN doc for Classes
@ntban15
Copy link
Contributor Author

ntban15 commented Feb 28, 2019

Mình vừa update từ 'Cảnh báo' thành 'Lưu ý' và link MDN doc cho Class bằng tiếng Việt.

Copy link
Collaborator

@nampdn nampdn left a comment

Choose a reason for hiding this comment

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

Cảm ơn @ntban15 nè!

@tesseralis tesseralis mentioned this pull request Feb 28, 2019
95 tasks
@ltmylinh
Copy link
Collaborator

ltmylinh commented Mar 2, 2019

Sorry mấy đứa, mấy nay chị busy xíu nên giờ mới vô translate tiếp được.


```js
<button onClick={(e) => this.deleteRow(id, e)}>Delete Row</button>
<button onClick={this.deleteRow.bind(this, id)}>Delete Row</button>
```

The above two lines are equivalent, and use [arrow functions](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions) and [`Function.prototype.bind`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_objects/Function/bind) respectively.
Hai dòng code trên là tương đương, và lần lượt sử dụng [hàm rút gọn](https://developer.mozilla.org/vi/docs/Web/JavaScript/Reference/Functions/Arrow_functions) [`Function.prototype.bind`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_objects/Function/bind).
Copy link
Collaborator

Choose a reason for hiding this comment

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

It should be: Hai dòng code trên là tương đương, và lần lượt sử dụng "hàm rút gọn" [arrow function]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Chỗ này ở trên có đề cập tới hàm rút gọn trước nên mình update ở đó nhé

+ Update title
+ Add some terms explanation
@ltmylinh ltmylinh removed the In Review label Mar 7, 2019
@ltmylinh ltmylinh merged commit 3d66a8f into reactjs:master Mar 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants