8000 Pick up some changes by re-fort · Pull Request #330 · vuejs/jp.vuejs.org · GitHub
[go: up one dir, main page]

Skip to content

Pick up some changes #330

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 7 commits into from
Sep 11, 2017
Merged

Pick up some changes #330

merged 7 commits into from
Sep 11, 2017

Conversation

re-fort
Copy link
Collaborator
@re-fort re-fort commented Sep 8, 2017

sibiraj-s and others added 6 commits September 8, 2017 22:26
* fix extract css related link

* change to secureable schema

# Conflicts:
#	src/v2/guide/deployment.md
As someone new to Vuejs, it took me a few extra seconds to grok the expanded code block for when `v-model` is used with a component. My mind was somehow trying to relate the 3rd block of code with the 2nd block, instead of the 1st.

(https://vuejs.org/v2/guide/components.html#Form-Input-Components-using-Custom-Events).

I suggest re-wording it to make it a bit more obvious.
# Conflicts:
#	src/v2/guide/components.md
# Conflicts:
#	themes/vue/layout/post.ejs
Remove extra comma in the Object Syntax example for computed properties
@re-fort re-fort requested review from kazupon and potato4d September 8, 2017 14:18
Copy link
Member
@kazupon kazupon left a comment

Choose a reason for hiding this comment

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

レビュー 👀 しました!
コメント 💬 の対応お願いします!

@@ -78,9 +78,9 @@ Webpack を使用している場合、JavaScript と テンプレートの分離

それぞれのビルドツールのドキュメントを参照してください:

- [Webpack + vue-loader](http://vue-loader.vuejs.org/en/configurations/extract-css.html) (`vue-cli` の webpack テンプレートは既に設定済み)
- [Webpack + vue-loader](https://vue-loader.vuejs.org/en/configurations/extract-css.html) (`vue-cli` の webpack テンプレートは既に設定済み)
Copy link
Member

Choose a reason for hiding this comment

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

日本語向けなので、下記日本語訳のvue-loaderのリンク先にしましょう!
https://vue-loader.vuejs.org/ja/configurations/extract-css.html

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ありがとうございます!修正しました!

Copy link
Member
@kazupon kazupon left a comment

Choose a reason for hiding this comment

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

👍

@kazupon kazupon merged commit a0908aa into vuejs:lang-ja Sep 11, 2017
@re-fort re-fort deleted the pick-up-some-changes branch September 12, 2017 02:09
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.

6 participants
0