-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
doc: translation render props #106
Conversation
Deploy preview for cn-reactjs ready! Built with commit d05c25a |
@chenshuai2144 有些需要调整哈 |
Deploy preview for zh-hans-reactjs ready! Built with commit d05c25a |
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.
原文还存在较多问题,建议 @QC-L 再添加一位 reviewer 进行审校。
- 「您」的称呼
- Google Translate 应该会默认略去 `` 符号,但翻译中需再添回
- 部分语句还有些不通畅
- 中英文空格问题
- 类似「复用」的词汇全文应保持一致
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.
一些小修改~
需要等等 最近有点忙 |
Co-Authored-By: LeoEatle <[email protected]>
谜之重复,我直接合了 Co-Authored-By: LeoEatle <[email protected]>
这种中英文空格问题我都直接合入 Co-Authored-By: LeoEatle <[email protected]>
Co-Authored-By: LeoEatle <[email protected]>
Co-Authored-By: LeoEatle <[email protected]>
Co-Authored-By: LeoEatle <[email protected]>
Co-Authored-By: LeoEatle <[email protected]>
QC-L 提出的修改大多数是中英文空格问题和格式问题,我都直接提交了。麻烦把 hijiangtao 提出的建议改一下,之后 @hijiangtao 看下修改没问题就能合入了。 刚才提交的时候应当用 batch 合并 commit 的,最后合并的时候注意要 squash and merge 。 |
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.
LGTM
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.
LGTM
translation render props