-
Notifications
You must be signed in to change notification settings - Fork 112
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
Conversation
Deploy preview for vi-reactjs-org ready! Built with commit 00cd756 |
There was a problem hiding this 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!!!
content/docs/faq-functions.md
Outdated
> | ||
>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). |
There was a problem hiding this comment.
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".
content/docs/faq-functions.md
Outdated
|
||
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` và `debounce`. |
There was a problem hiding this comment.
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ụ"
There was a problem hiding this comment.
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
* 'master' of https://github.com/reactjs/vi.reactjs.org: resolve conflict Some fixes to the Accessibility guide (#2047)
@ltmylinh Pls, Review it ! |
Thanks @cuongtran8, i fixed and PR, review it. |
There was a problem hiding this 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
* '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)
There was a problem hiding this 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!
@ltmylinh Merge it, tks |
No description provided.