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 learn/you-might-not-need-an-effect into Chinese #1155

Merged
merged 22 commits into from
Jun 21, 2023

Conversation

shezhangzhang
Copy link
Contributor

@shezhangzhang shezhangzhang commented May 4, 2023

This is my first time translating React documentation into Chinese, and I have tried my best to do so. The article I translated was quite huge in this PR, it was an uphill battle. I have spent a lot of time on it. Hope it helps!

这是我第一次翻译 React 文档的 PR,哈哈,尽了最大的努力。这个PR中我翻译的这篇文章非常非常长,我花了很多时间来做。我已经检查过一遍,辛苦审阅,谢谢!

@QC-L @Yucohny @awxiaoxian2020 @Huxpro

@github-actions
Copy link

github-actions bot commented May 4, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

Three Pages Changed Size

The following pages changed size from the code in this PR compared to its base branch:

Page Size (compressed) First Load
/404 76.66 KB (🟡 +9 B) 179.82 KB
/500 76.66 KB (🟡 +9 B) 179.81 KB
/[[...markdownPath]] 78.23 KB (🟡 +9 B) 181.39 KB
Details

Only the gzipped size is provided here based on an expert tip.

First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If next/link is used, subsequent page loads would only need to download that page's bundle (the number in the "Size" column), since the global bundle has already been downloaded.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

Next to the size is how much the size has increased or decreased compared with the base branch of this PR. If this percentage has increased by 10% or more, there will be a red status indicator applied, indicating that special attention should be given to this.

@shezhangzhang
Copy link
Contributor Author

更新:将所有导航(footer and slidebar)的 escape hatches 翻译从 应急方案 修改成了 逃脱方案。讨论见:此处

aimer-fan
aimer-fan previously approved these changes May 15, 2023
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
src/content/learn/you-might-not-need-an-effect.md Outdated Show resolved Hide resolved
Co-authored-by: Xleine <919143384@qq.com>
shezhangzhang and others added 2 commits May 23, 2023 11:14
Co-authored-by: Xleine <919143384@qq.com>
Co-authored-by: Xleine <919143384@qq.com>
Co-authored-by: Xleine <919143384@qq.com>
shezhangzhang and others added 4 commits May 23, 2023 11:21
Co-authored-by: Xleine <919143384@qq.com>
Co-authored-by: Xleine <919143384@qq.com>
Co-authored-by: Xleine <919143384@qq.com>
Co-authored-by: Xleine <919143384@qq.com>
@shezhangzhang
Copy link
Contributor Author

@loveloki Thank you very much, Great feedback! 谢谢!

@loveloki
Copy link
Collaborator

@loveloki Thank you very much, Great feedback! 谢谢!

客气,我提供的修改也不一定会更好,你还是要自己判断是否应该修改。我今晚继续往下看

@shezhangzhang
Copy link
Contributor Author

shezhangzhang and others added 2 commits June 19, 2023 10:58
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
shezhangzhang and others added 3 commits June 19, 2023 11:01
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
Co-authored-by: Yucohny <79147654+Yucohny@users.noreply.github.com>
Co-authored-by: Xleine <919143384@qq.com>
@shezhangzhang
Copy link
Contributor Author

@Yucohny 全部搞定了,谢谢!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending Review 已翻译,待校对阶段
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants