Skip to content
New issue

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

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

Already on GitHub? # to your account

docs(cn): translate content/docs/forwarding-refs.md into Chinese #118

Merged
merged 10 commits into from
Apr 15, 2019

Conversation

yhsunshining
Copy link
Contributor

Translate content/docs/forwarding-refs.md into Chinese.

@netlify
Copy link

netlify bot commented Feb 18, 2019

Deploy preview for cn-reactjs ready!

Built with commit 13d2793

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

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

@robertying robertying left a comment

Choose a reason for hiding this comment

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

赞👍!
指出了一些小错误,提供了一部分额外建议。还需要对某些格式问题做约定……

引号和英文空格问题可以先无视,麻烦啦!

content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
examples/forwarding-refs/fancy-button-ref.js Outdated Show resolved Hide resolved
examples/forwarding-refs/log-props-after.js Outdated Show resolved Hide resolved
examples/forwarding-refs/log-props-after.js Outdated Show resolved Hide resolved
examples/forwarding-refs/log-props-after.js Outdated Show resolved Hide resolved
examples/forwarding-refs/log-props-after.js Outdated Show resolved Hide resolved
…description to make the doc easier to understand.
@QC-L QC-L added Pending Re-Review 已修改,待审校阶段 and removed Pending Review 已翻译,待校对阶段 labels Feb 21, 2019
2. Update glossary translation.
3. Update format.
@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for zh-hans-reactjs ready!

Built with commit 13d2793

https://deploy-preview-118--zh-hans-reactjs.netlify.com

content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved

For example, the following component will appear as "*ForwardRef*" in the DevTools:
示例,以下组件将在 DevTools 中显示为 “*ForwardRef*”:
Copy link
Member

Choose a reason for hiding this comment

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

中文全角符号两侧不需要空格。

Suggested change
示例,以下组件将在 DevTools 中显示为 *ForwardRef*”:
示例,以下组件将在 DevTools 中显示为“*ForwardRef*”:

content/docs/forwarding-refs.md Show resolved Hide resolved
@yhsunshining
Copy link
Contributor Author

@robertying @yuqingc 感谢两位大佬~
相关修改已经完成了,为了保证之后的样式,这里讨论确认后,采用了全角标点+空格的形式。两位大佬有时间可以看下还有什么修改建议。
再次感谢~

robertying
robertying previously approved these changes Apr 12, 2019
yuqingc
yuqingc previously approved these changes Apr 12, 2019
Copy link
Collaborator

@Lmagic16 Lmagic16 left a comment

Choose a reason for hiding this comment

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

some suggestion


**When you start using `forwardRef` in a component library, you should treat it as a breaking change and release a new major version of your library.** This is because your library likely has an observably different behavior (such as what refs get assigned to, and what types are exported), and this can break apps and other libraries that depend on the old behavior.
**当你开始在组件库中使用 `forwardRef` 时,你应当将其作为一个破坏性更改,并发布库的一个新的主板本。** 这是因为你的库可能会有明显不同的行为(例如 refs 被分配给了什么,以及导出了什么类型),并且这样可能会导致依赖旧行为的应用和其他库崩溃。
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**当你开始在组件库中使用 `forwardRef` 时,你应当将其作为一个破坏性更改,并发布库的一个新的主板本** 这是因为你的库可能会有明显不同的行为(例如 refs 被分配给了什么,以及导出了什么类型),并且这样可能会导致依赖旧行为的应用和其他库崩溃。
**当你开始在组件库中使用 `forwardRef` 时,你应当将其视为一个破坏性更改,并发布库的一个新的主版本** 这是因为你的库可能会有明显不同的行为(例如 refs 被分配给了谁,以及导出了什么类型),并且这样可能会导致依赖旧行为的应用和其他库崩溃。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what refs get assigned to
字面上理解 应该是指什么, 但是此处用谁好像更通顺一点。但在不影响表意的情况下,个人倾向于更遵照直译,大佬怎么看?

Copy link
Collaborator

Choose a reason for hiding this comment

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

额 这里是第一次读的时候觉得有点怪,不过确实不影响表意。遵照直译吧~

Copy link
Collaborator

Choose a reason for hiding this comment

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

这里需要注意一下“主板本”=>“主版本”

Copy link
Member

Choose a reason for hiding this comment

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

@yhsunshining 主版本改下

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@QC-L @Lmagic16 这里都修改完了~

content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
content/docs/forwarding-refs.md Outdated Show resolved Hide resolved
2. Modify part description.
@yhsunshining yhsunshining dismissed stale reviews from yuqingc and robertying via 8d8aa35 April 13, 2019 09:27
QC-L
QC-L previously approved these changes Apr 14, 2019
yuqingc
yuqingc previously approved these changes Apr 15, 2019
Copy link
Member

@yuqingc yuqingc left a comment

Choose a reason for hiding this comment

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

ok

@yhsunshining yhsunshining dismissed stale reviews from yuqingc and QC-L via 13d2793 April 15, 2019 04:29
@QC-L QC-L merged commit ea0be44 into reactjs:master Apr 15, 2019
OhIAmFine pushed a commit to OhIAmFine/zh-hans.reactjs.org that referenced this pull request May 6, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Pending Re-Review 已修改,待审校阶段
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants