Skip to content
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

docs(cn): translate content/warnings/unknown-prop.md into Chinese #111

Merged
merged 4 commits into from Feb 24, 2019
Merged

docs(cn): translate content/warnings/unknown-prop.md into Chinese #111

merged 4 commits into from Feb 24, 2019

Conversation

seognil
Copy link
Contributor

@seognil seognil commented Feb 17, 2019

https://stackblitz.com/edit/react-unknown-prop
无法重现这个 warning 可能是我例子写得不够高级…

@netlify
Copy link

netlify bot commented Feb 17, 2019

Deploy preview for cn-reactjs ready!

Built with commit 713848f

https://deploy-preview-111--cn-reactjs.netlify.com

@seognil seognil mentioned this pull request Feb 17, 2019
@QC-L QC-L added the Pending Review 已翻译,待校对阶段 label Feb 17, 2019

---

To fix this, composite components should "consume" any prop that is intended for the composite component and not intended for the child component. Example:
要解决这个问题,组合组件应该“消耗”任何用于复合组件的 prop,而不要用于子组件。例如:
Copy link
Collaborator

@LeoEatle LeoEatle Feb 18, 2019

Choose a reason for hiding this comment

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

Suggested change
要解决这个问题,组合组件应该“消耗”任何用于复合组件的 prop,而不要用于子组件。例如:
要解决这个问题,组合组件应该“处理”任何只用于复合组件的 prop,而不要用于子组件。例如:

感觉用“处理”会比“消耗”更好理解,或许用“剥离”也可以,待讨论
加个“只”更有助于理解为什么不能让多余的props传入子组件。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改进了一下(这句有点头疼

content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
@QC-L QC-L added Pending Modify 已校对,待修改阶段 and removed Pending Review 已翻译,待校对阶段 labels Feb 18, 2019
Copy link
Member

@hijiangtao hijiangtao left a comment

Choose a reason for hiding this comment

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

  1. 文中关于 Good / Bad 需要确认下是否为通用译法 @QC-L
  2. 译文需检查一遍,保持与原文行数一致
  3. 部分建议见 comments

感谢参与翻译工作!

content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
content/warnings/unknown-prop.md Outdated Show resolved Hide resolved
LeoEatle
LeoEatle previously approved these changes Feb 20, 2019
Copy link
Collaborator

@LeoEatle LeoEatle left a comment

Choose a reason for hiding this comment

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

Approved

@hijiangtao hijiangtao added Pending Re-Review 已修改,待审校阶段 and removed Pending Modify 已校对,待修改阶段 labels Feb 24, 2019
@hijiangtao hijiangtao requested a review from QC-L February 24, 2019 03:03
@QC-L QC-L merged commit 0e8e1b7 into reactjs:master Feb 24, 2019
zhaozhiming pushed a commit to zhaozhiming/zh-hans.reactjs.org that referenced this pull request Feb 25, 2019
OhIAmFine pushed a commit to OhIAmFine/zh-hans.reactjs.org that referenced this pull request May 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending Re-Review 已修改,待审校阶段
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants