Skip to content

Translate FAQ/Function and FAQ/Styling #38

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 10 commits into from
Jun 18, 2019
Merged

Conversation

tuantvk
Copy link
Contributor

@tuantvk tuantvk commented Jun 4, 2019

No description provided.

@netlify
Copy link

netlify bot commented Jun 4, 2019

Deploy preview for vi-reactjs-org ready!

Built with commit 00cd756

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

Copy link
Collaborator

@ltmylinh ltmylinh left a comment

Choose a reason for hiding this comment

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

Everything looks good to me. a minor change. Thanks @tuantvk!!!

>
>Using `Function.prototype.bind` in render creates a new function each time the component renders, which may have performance implications (see below).
>Sử dụng `Function.prototype.bind` trong hàm render tạo ra một function mới mỗi lần component renders, điều đó có thể liên quan tới hiệu xuất (Xem bên dưới).
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 "hiệu suất".


See [this visualization](http://demo.nimius.net/debounce_throttle/) for a comparison of `throttle` and `debounce` functions.
Xem [ví dụ này](http://demo.nimius.net/debounce_throttle/) để so sánh giữa các hàm `throttle` `debounce`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

visualization mình có thể translate là "đặc tả" hay gì đó được hơn là "ví dụ"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mình nghĩ dịch thành Xem mô phỏng này có vẻ hợp lý hơn

@tuantvk
Copy link
Contributor Author

tuantvk commented Jun 5, 2019

@ltmylinh Pls, Review it !

@tuantvk
Copy link
Contributor Author

tuantvk commented Jun 5, 2019

Thanks @cuongtran8, i fixed and PR, review it.

Copy link
Collaborator

@cuongtran8 cuongtran8 left a comment

Choose a reason for hiding this comment

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

Một vài chỗ nhỏ cần sửa chút, xin lỗi bạn sáng nay đang review mà chạy vô họp nên chưa review hết

tuantvk added 2 commits June 6, 2019 10:06
* 'master' of https://github.com/reactjs/vi.reactjs.org:
  Remove outdated link to React FAQ website (#2043)
  Use imperative tone for suggested commit message (#2050)
Copy link
Collaborator

@ltmylinh ltmylinh left a comment

Choose a reason for hiding this comment

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

Everything looks good to me! Thanks @tuantvk!

@cuongtran8
Copy link
Collaborator

Thank a lot @tuantvk , really good job!!!!!! thank @ltmylinh for review also :D

@tuantvk
Copy link
Contributor Author

tuantvk commented Jun 17, 2019

@ltmylinh Merge it, tks

@cuongtran8 cuongtran8 merged commit 4897b26 into reactjs:master Jun 18, 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.

3 participants