Skip to content
This repository was archived by the owner on Aug 8, 2022. It is now read-only.

style: Unified style #203

Merged
merged 6 commits into from
Oct 20, 2020
Merged

style: Unified style #203

merged 6 commits into from
Oct 20, 2020

Conversation

FAKER-A
Copy link
Contributor

@FAKER-A FAKER-A commented Oct 19, 2020

Description of Problem

Proposed Solution

Additional Information

@veaba
Copy link
Member

veaba commented Oct 19, 2020

感谢你的 PR,前面第一个 commit,尽管这是一个很小的语法错误,但检查到英文版没有这个改动,建议你先去英文版发起这个改动后中文版才会合并。

这个 PR 暂时挂起。

下面提出了一些修改意见~~

@FAKER-A
Copy link
Contributor Author

FAKER-A commented Oct 19, 2020

感谢你的 PR,前面第一个 commit,尽管这是一个很小的语法错误,但检查到英文版没有这个改动,建议你先去英文版发起这个改动后中文版才会合并。

这个 PR 暂时挂起。

好的, 感谢回复! 第一个commit已合并到英文版。vuejs/docs#636 (comment)

@veaba
Copy link
Member

veaba commented Oct 19, 2020

@FAKER-A

关于加粗符号内的符号使用,请参考我们之前讨论的 #78 。(这里影响翻译的迁移到右侧半角符号转换全角符号的 wiki

另外稍微麻烦下,后面的更改顺便把 <sub> 标签的内容移除下,尽量保持原本一致,这是历史遗留的翻译所致。

关于 “breaking” 的问题见 #62

@@ -49,7 +49,7 @@ Vue.directive('highlight', {
- **beforeUpdate**:<sup style="color:green">新的!</sup>这是在元素本身更新之前调用的,很像组件生命周期钩子
- update → <sup style="color:red">移除!</sup>有太多的相似之处要更新,所以这是多余的,请改用 `updated`
- componentUpdated → **updated**
- **beforeUnmount** <sup style="color:green">新的!</sup>与组件生命周期钩子类似,它将在卸载元素之前调用。
- **beforeUnmount**:<sup style="color:green">新的!</sup>与组件生命周期钩子类似,它将在卸载元素之前调用。
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
- **beforeUnmount**:<sup style="color:green">新的!</sup>与组件生命周期钩子类似,它将在卸载元素之前调用。
- **beforeUnmount**新的!与组件生命周期钩子类似,它将在卸载元素之前调用。

- **BREAKING:**自定义元素白名单现在在模板编译期间执行,应该通过编译器选项而不是运行时配置来配置。
- **BREAKING:**特定 `is` prop 用法仅限于保留的 `<component>` 标记
- **NEW:**有了新的 `v-is` 指令来支持 2.x 用例,其中在原生元素上使用了 `v-is` 来处理原生 HTML 解析限制。
- **BREAKING:** 自定义元素白名单现在在模板编译期间执行,应该通过编译器选项而不是运行时配置来配置。
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
- **BREAKING:** 自定义元素白名单现在在模板编译期间执行,应该通过编译器选项而不是运行时配置来配置。
- **非兼容**自定义元素白名单现在在模板编译期间执行,应该通过编译器选项而不是运行时配置来配置。

Copy link
Member

Choose a reason for hiding this comment

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

see #62

- **BREAKING:**特定 `is` prop 用法仅限于保留的 `<component>` 标记
- **NEW:**有了新的 `v-is` 指令来支持 2.x 用例,其中在原生元素上使用了 `v-is` 来处理原生 HTML 解析限制。
- **BREAKING:** 自定义元素白名单现在在模板编译期间执行,应该通过编译器选项而不是运行时配置来配置。
- **BREAKING:** 特定 `is` prop 用法仅限于保留的 `<component>` 标记
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
- **BREAKING:** 特定 `is` prop 用法仅限于保留的 `<component>` 标记
- **非兼容**特定 `is` prop 用法仅限于保留的 `<component>` 标记

Copy link
Member

Choose a reason for hiding this comment

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

see #62

- **NEW:**有了新的 `v-is` 指令来支持 2.x 用例,其中在原生元素上使用了 `v-is` 来处理原生 HTML 解析限制。
- **BREAKING:** 自定义元素白名单现在在模板编译期间执行,应该通过编译器选项而不是运行时配置来配置。
- **BREAKING:** 特定 `is` prop 用法仅限于保留的 `<component>` 标记
- **NEW:** 有了新的 `v-is` 指令来支持 2.x 用例,其中在原生元素上使用了 `v-is` 来处理原生 HTML 解析限制。
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
- **NEW:** 有了新的 `v-is` 指令来支持 2.x 用例,其中在原生元素上使用了 `v-is` 来处理原生 HTML 解析限制。
- ****有了新的 `v-is` 指令来支持 2.x 用例,其中在原生元素上使用了 `v-is` 来处理原生 HTML 解析限制。

Copy link
Member

Choose a reason for hiding this comment

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

see #62

@FAKER-A
Copy link
Contributor Author

FAKER-A commented Oct 20, 2020

@FAKER-A

关于加粗符号内的符号使用,请参考我们之前讨论的 #78 。(这里影响翻译的迁移到右侧半角符号转换全角符号的 wiki

另外稍微麻烦下,后面的更改顺便把 <sub> 标签的内容移除下,尽量保持原本一致,这是历史遗留的翻译所致。

关于 “breaking” 的问题见 #62

已针对上述问题进行了修改。github用的不太熟练,还请见谅:flushed:

@veaba veaba merged commit f618b60 into vuejs:master Oct 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants