-
Notifications
You must be signed in to change notification settings - Fork 8.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
[zh-cn]: add View Transitions API document translation #15643
Conversation
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.
Hey @suiyun39, I've listed some common issue in this PR. Please check all the files and apply a full correction.
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.
suggestions for the first two documents
GitHub has provide UI to apply all the suggestions in one commit, which would be better for the reviewer to check feedbacks: https://docs.github.com/zh/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/incorporating-feedback-in-your-pull-request You could try this feature in files for the coming suggestions. |
This feature is really great, thanks for the reminder. |
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.
next three documents
Co-authored-by: A1lo <[email protected]>
files/zh-cn/web/css/_doublecolon_view-transition-image-pair/index.md
Outdated
Show resolved
Hide resolved
files/zh-cn/web/css/_doublecolon_view-transition-group/index.md
Outdated
Show resolved
Hide resolved
files/zh-cn/web/css/_doublecolon_view-transition-group/index.md
Outdated
Show resolved
Hide resolved
files/zh-cn/web/css/_doublecolon_view-transition-image-pair/index.md
Outdated
Show resolved
Hide resolved
files/zh-cn/web/css/_doublecolon_view-transition-image-pair/index.md
Outdated
Show resolved
Hide resolved
Co-authored-by: A1lo <[email protected]>
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.
Thanks @suiyun39 💯
Thanks also for your review and suggestions. Regards. |
Description
Add View Transitions API document translation of zh-cn
Motivation
Additional details
Related issues and pull requests