Skip to content

docs: update options-api.md #11

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

Merged
merged 9 commits into from
Feb 18, 2022
Merged

docs: update options-api.md #11

merged 9 commits into from
Feb 18, 2022

Conversation

wxsms
Copy link
Member

@wxsms wxsms commented Jan 16, 2022

Description of Problem

typescript/options-api.md

@wxsms wxsms added the 校对 label Jan 16, 2022
@wxsms wxsms marked this pull request as ready for review January 16, 2022 06:38
wxsms added 2 commits January 16, 2022 14:53
# Conflicts:
#	src/guide/typescript/options-api.md
@@ -92,9 +92,9 @@ const Component = defineComponent({

这会防止 Typescript 将 `this` 根据函数内的环境作出不符合我们期望的类型推导。

## 为组件的 emits 标注类型 {#typing-component-emits}
## 为组件的 emit 标注类型 {#typing-component-emits}
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
## 为组件的 emit 标注类型 {#typing-component-emits}
## 为组件的 emits 标注类型 {#typing-component-emits}

Copy link
Member

Choose a reason for hiding this comment

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

标题明确指向的是 emits 而非正常语言,所以这里保留原文。

Copy link
Member Author

@wxsms wxsms Feb 9, 2022

Choose a reason for hiding this comment

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

这里我也有所疑惑。但是按照我们目前的翻译须知:

中文翻译中夹带的英文一律以单数形式呈现

因此可能有以下几种解决方案:

  1. 如果需要保留复数,按照目前的规则,可以:
    1. 通过加 code style,加 code style 的话也应该要向英文版反馈,同步加上(edit:evan 建议不要为该 title 加codestyle,见回复
    2. 通过修改我们的翻译须知,为有”明确指向“的单词保留复数(但是哪些情况有“明确指向”,可能也需要讨论)
  2. 遵守翻译须知,使用单数

@veaba veaba merged commit 174d851 into vuejs-translations:main Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants