Skip to content

[docs][zh-cn] synced recent updates #429

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 2 commits into from
Feb 22, 2018
Merged

[docs][zh-cn] synced recent updates #429

merged 2 commits into from
Feb 22, 2018

Conversation

Jinjiang
Copy link
Member

Ref: 865c526...6fa6e06
/ping @fnlctrl
Thanks.

@eddyerburgh
Copy link
Member

Thanks @Jinjiang . Could you also remove Chai references from the chinese docs 🙏, like in this PR—#439

@Jinjiang
Copy link
Member Author

No problem.👌
Will do that soon.

@@ -18,6 +18,8 @@ shallow(Component, {
})
```

> **注意:**在一个 `localVue` 上安装 Vue Router 时也会将 `$route` 和 `$router` 作为两个只读属性添加给该 `localVue`。这意味着如果你使用安装了 Vue Router 的 `localVue`,则不能在挂在一个组件时使用 `mocks` 选项来复写 `$route` 和 `$router`。
Copy link
Member

Choose a reason for hiding this comment

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

复写 => 覆写

Copy link
Member Author

Choose a reason for hiding this comment

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

@fnlctrl 我检查了一下,这个词我在别的一些地方也用了“复写” (如 config.md、optinons.md 还有 vue-cli、style-guide 等地方),另外我的输入法、百度关键字推荐等地方似乎都只有“复写”这个词。这里是否可以保持“复写”?🙂
谢谢

Copy link
Member

Choose a reason for hiding this comment

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

抱歉这两天没看通知:joy:

复写: 把复写纸夹在两张或几张纸之间书写,能写成一式多份(http://www.zdic.net/c/d/152/339032.htm, https://baike.baidu.com/item/%E5%A4%8D%E5%86%99)
英语中对应的是Copywriting
image

汉语词典本身没有“覆写”这个词,"覆"也没有"复"常用,所以输入法没有收录。但鉴于over是覆盖的意思,用"覆"可能更为恰当
image

Copy link
Member Author

Choose a reason for hiding this comment

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

认同。感谢指正,我稍后全局替换一下。

Copy link
Member Author

Choose a reason for hiding this comment

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

@Jinjiang
Copy link
Member Author

@eddyerburgh @fnlctrl Btw, seems this PR has been reviewed and is close to be approved, I'd like to create another PR separately for removing Chai ref if you two don't mind :-)

@eddyerburgh
Copy link
Member

Yep that's fine, thanks @Jinjiang @fnlctrl 🙌

@eddyerburgh eddyerburgh merged commit 871a582 into vuejs:dev Feb 22, 2018
@Jinjiang Jinjiang deleted the zh-cn branch February 23, 2018 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants