Skip to content

Update #860

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 53 commits into from
Feb 13, 2019
Merged

Update #860

merged 53 commits into from
Feb 13, 2019

Conversation

Jinjiang
Copy link
Member

@Jinjiang Jinjiang commented Jan 30, 2019

Ref: vuejs/v2.vuejs.org@242dea4...1a73684

except for:

Thanks.

hudarui and others added 30 commits January 30, 2019 11:13
Unnecessary escape character: \-
Otherwise, VNode is used without definition the first time you encounter
it in the Guide.

Fixes #1799.
So they don't require horizontal scrolling
* Unify examples HTML formatting

* Unify examples HTML formatting
* docs: Add Ben Hong to Community Partners

* style: Remove new lines from autoformatting
* Adds Code Of Conduct via contributor-covenant version 1.4

* Change community email

* Resolve conflict
[List Rendering](https://vuejs.org/v2/guide/list.html#key) guide doesn't mention what types of values user can supply for `key` of `v-for`. There is a console warning for this already but it's hard to point people to it. This pull request fixes the issue.
`$attrs` and `$listeners` were introduced in v2.4.0. The docs should mention this in order to avoid confusion.
* Add Vuejs Amsterdam

* fix profiles for vue.js amsterdam organizers
* Update dockerize cookbook

Node 9 has been EOL for a while, this will make sure people will always use the latest LTS version, instead of a hardcoded version number

* Update dockerize-vuejs-app.md
The link to the live demo is not up-to-date. I've updated it to the same one as noted in [vue-hackernews-2.0](https://github.com/vuejs/vue-hackernews-2.0/blob/master/README.md)
yyx990803 and others added 18 commits January 30, 2019 13:09
This commit fixes broken instructions on how to bundle the Runtime + Compiler with Rollup.

Related: rollup/rollup#2609
The example of using async_function syntax is wrong, fix it.
Removing Vuetron as the project is no longer active.
* Clarify that "slot-scope" is named "scope" before 2.5.0

* Improve wording
Fixes [#1765](vuejs/v2.vuejs.org#1765).

I left the "define custom key modifier aliases" section at the bottom, below the deprecated key codes information. Definitely open to changing / re-arranging as needed!
* vuejs-templates/* is deprecated because of Vue CLI 3
* vue-touch has been depreceated because of lack of development
@Jinjiang Jinjiang requested a review from Justineo January 30, 2019 07:36
@Jinjiang
Copy link
Member Author

Also, some of the images in /theme/vue/source/images has been removed because they seem not useful anymore:

  • coding-net-ad.jpg
  • fancygrid.png
  • gitee_ad.png
  • geekbang.png

@yyx990803 you may need to have a check because it's sponsors related.

Thanks.

@Jinjiang Jinjiang requested a review from yyx990803 January 30, 2019 07:41
Justineo and others added 5 commits January 30, 2019 16:17
Co-Authored-By: Jinjiang <zhaojinjiang@me.com>
Co-Authored-By: Jinjiang <zhaojinjiang@me.com>
Co-Authored-By: Jinjiang <zhaojinjiang@me.com>
Co-Authored-By: Jinjiang <zhaojinjiang@me.com>
@Jinjiang
Copy link
Member Author

  • vuejs/v2.vuejs.org@0c1ff02 All videos in Vue Mastery are deployed on Vimeo which is blocked in CN. And also the script from vimeo.com would probably block the page loading. So I didn't sync this change. @yyx990803 is that OK to drop this commit here?

Also, some of the images in /theme/vue/source/images has been removed because they seem not useful anymore:

  • coding-net-ad.jpg
  • fancygrid.png
  • gitee_ad.png
  • geekbang.png

@yyx990803 you may need to have a check because it's sponsors related.

Memo: these two things above have already been orally agreed by @yyx990803. So merged.

@Jinjiang Jinjiang merged commit 8c00afb into master Feb 13, 2019
@Jinjiang Jinjiang deleted the update branch February 13, 2019 01:07
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.