Update reactivity.md #697
Merged
Update reactivity.md #697
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
前一半 LGTM。 后面一半感觉原文挺清晰的呀？ |
Jinjiang
reviewed
Apr 3, 2018
src/v2/guide/reactivity.md
Outdated
| @@ -70,7 +70,7 @@ var vm = new Vue({ | |||
| vm.message = 'Hello!' | |||
| ``` | |||
|
|
|||
| 如果你在 data 选项中未声明 `message`，Vue 将警告你渲染函数在试图访问的属性不存在。 | |||
| 如果你未在 data 选项中声明 `message`，Vue 会抛出向你一条警告：render 函数正在试图访问的一个属性，在实例中并不存在。 | |||
There was a problem hiding this comment.
“……Vue 将警告你渲染函数正在试图访问一个不存在的属性。”这样更好一些吗？
Btw. 这里原文的 render function 我觉得应该翻译成“渲染函数”，而不是特指 render 这个函数名。如果是后者的话，原文这里会用反引号括起来。
谢谢
There was a problem hiding this comment.
其实我想强调的是正在和在实例 data 中，我修正一下。
Justineo
reviewed
Apr 3, 2018
src/v2/guide/reactivity.md
Outdated
| @@ -70,7 +70,7 @@ var vm = new Vue({ | |||
| vm.message = 'Hello!' | |||
| ``` | |||
|
|
|||
| 如果你在 data 选项中未声明 `message`，Vue 将警告你渲染函数在试图访问的属性不存在。 | |||
| 如果你未在 data 选项中声明 `message`，Vue 将警告你渲染函数正在试图访问的属性在实例的 data 中不存在。 | |||
There was a problem hiding this comment.
加上 data 是不对的，因为 Vue 并不会知道你的数据会在 data/props/computed 中的哪一个。Vue 只能知道不存在。
There was a problem hiding this comment.
想了想这个案例是以 data 举例，说在实例的 data 中也是可以，不过想想还是改回原译吧。
Jinjiang
approved these changes
Apr 3, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.