-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Chinese version translated/翻译了中文版本 #1057
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
Conversation
Chinese version/中文版本
CLA assistant check
All committers have signed the CLA.
✅ Deploy Preview for jovial-austin-42fe02 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Can you please sign the CLA?
...ite_and_docs/content/documentation/webdriver/bidirectional/bidi_api_remotewebdriver.zh-cn.md
Outdated
Show resolved
Hide resolved
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.
Can you keep the WebDriver as a Proper Noun ?
...ite_and_docs/content/documentation/webdriver/bidirectional/bidi_api_remotewebdriver.zh-cn.md
Outdated
Show resolved
Hide resolved
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.
It's not monitor only.
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.
I will change then translation.I think should use '截网侦察'
Thanks for your first contribution. Just leave some comments, please feel free to discuss.
Generally, it's the "Improved translation", you should keep the wording under same translation in one post, and meanwhile, the conversion is better to avoid long sentence in one line. At last, keep original format(e.g. command) and punctuation(e.g. half-width symbol) is friend to validate the layout.
Modified some translations。
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.
Where is BiDi APIs ?
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.
There is a code , please keep format as previous mentioned
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.
Almost are fine, only need aware the missed parts as mentioned.
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.
Hi, will you do the post commit to avoid impact original description and code segment?
Uh oh!
There was an error while loading. Please reload this page.
Chinese version/中文版本
Thanks for contributing to the Selenium site and documentation!
A PR well described will help maintainers to quickly review and merge it
Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.
Description
Motivation and Context
Types of changes
Checklist