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.

Sign up for GitHub

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

Jump to bottom

fix: change some details for ReactDOM #901

Merged
merged 2 commits into from
Feb 15, 2023
Merged

fix: change some details for ReactDOM #901

merged 2 commits into from
Feb 15, 2023

Conversation

hexiaokang
Copy link
Contributor

No description provided.

@hexiaokang
Copy link
Contributor Author

1、第43行
原英文：React supports all modern browsers, although some polyfills are required for older versions.
原翻译：React 支持所有的现代浏览器，尽管对于旧版本来说，可能需要引入 相关的 polyfills 依赖。
修改意见：此处的语境应该是转折，个人建议although翻译成‘不过’会更通顺些。

2、第47行的polyfill少了s

3、第65行的‘立即 更新’中间没必要有空格

4、第127行
原英文：because in most apps, mismatches are rare
原翻译：因为大多是应用中不匹配的情况很少见
修改意见：此处翻译为‘大多数’更符合语义

@hexiaokang hexiaokang changed the title 修改ReactDOM细节翻译 fix: change some details for ReactDOM Sep 4, 2022
@awxiaoxian2020 awxiaoxian2020 added the Pending Modify 已校对，待修改阶段 label Feb 14, 2023
@awxiaoxian2020 awxiaoxian2020 added Pending Re-Review 已修改，待审校阶段 and removed Pending Modify 已校对，待修改阶段 labels Feb 14, 2023
@github-actions
Copy link

Size Changes

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

This PR introduced no changes to the javascript bundle 🙌

@daochouwangu daochouwangu added Pending merge 可以考虑合并 and removed Pending Re-Review 已修改，待审校阶段 labels Feb 14, 2023
@awxiaoxian2020 awxiaoxian2020 merged commit b6f0cab into reactjs:main Feb 15, 2023
@awxiaoxian2020 awxiaoxian2020 removed the Pending merge 可以考虑合并 label Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daochouwangu daochouwangu daochouwangu approved these changes

@awxiaoxian2020 awxiaoxian2020 awxiaoxian2020 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hexiaokang @daochouwangu @awxiaoxian2020