Skip to content

docs: update docs/ja #533

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 3 commits into from
Apr 26, 2018
Merged

docs: update docs/ja #533

merged 3 commits into from
Apr 26, 2018

Conversation

38elements
Copy link
Contributor

No description provided.

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.

レビュー 👀 しました。コメント 💬 確認お願いします!

@@ -1,4 +1,4 @@
# mount(component {, options}])
# mount(component [, options])
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
Contributor Author

Choose a reason for hiding this comment

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

原文が間違っているのでこのpull requestで修正しています。

@@ -37,7 +38,8 @@ expect(wrapper.is(Component)).toBe(true)

- 型: `{ [name: string]: Array<Component>|Component|string }`

コンポーネントにスロットコンテンツのオブジェクトを渡します。key はスロット名に対応します。値は、コンポーネント、コンポーネントの配列、またはテンプレート文字列のいずれかになります。
コンポーネントにスロットコンテンツのオブジェクトを渡します。key はスロット名に対応します。
値は、コンポーネント、コンポーネントの配列、またはテンプレート文字列のいずれかになります。
Copy link
Member

Choose a reason for hiding this comment

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

原文に合わせて、1行にしておいた方がいいと思います。

- 型: `{ [name: string]: string }`

コンポーネントにスコープ付きスロットのコンテンツのオブジェクトを渡します。key はスロット名に対応します。
値はテンプレート文字列を指定します。
Copy link
Member

Choose a reason for hiding this comment

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

ここも原文と合わせて1行にしておいた方がいいですね。
https://github.com/vuejs/vue-test-utils/blame/dev/docs/en/api/options.md#L84


* `<template>` タグを `scopedSlots` オプションのルート要素として使用することはできません。

* PhantomJS をサポートしません。代わりに [Puppeteer](https://github.com/karma-runner/karma-chrome-launcher#headless-chromium-with-puppeteer) を使用してください。
Copy link
Member

Choose a reason for hiding this comment

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

原文では改行しているのですが、どちらかに統一した方がいいですね。どちらが正しいのか迷わないので。


### `stubs`

- type: `{ [name: string]: Component | boolean } | Array<string>`

子のコンポーネントをスタブします。スタブまたはオブジェクトに対するコンポーネント名の配列になります。`stubs` が配列の場合、すべてのスタブは `<!---->` になります。
子コンポーネントをスタブします。スタブまたはオブジェクトに対するコンポーネント名の配列になります。
Copy link
Member

Choose a reason for hiding this comment

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

ここの行もそうですね。原文、翻訳言語どちらが正しいのか少し混乱します。
https://github.com/vuejs/vue-test-utils/blame/dev/docs/en/api/options.md#L102

@@ -108,7 +136,8 @@ expect(wrapper.vm.$route.path).toBe($route.path)

- 型: `Vue`

コンポーネントのマウント時に使用する [createLocalVue](./createLocalVue.md) によって作成された Vue のローカルコピーです。この Vue のコピーにプラグインをインストールすると、元の `Vue` コピーが汚染されなくなります。
コンポーネントのマウント時に使用する [createLocalVue](./createLocalVue.md) によって作成された Vue のローカルコピーです。
Copy link
Member

Choose a reason for hiding this comment

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

ここもそうです。原文では改行していないです。
https://github.com/vuejs/vue-test-utils/blame/dev/docs/en/api/options.md#L145

@@ -1,4 +1,4 @@
# shallow(component {, options}])
# shallow(component [, options])
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
Contributor Author

Choose a reason for hiding this comment

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

原文が間違っているのでこのpull requestで修正しています。

@@ -1,4 +1,4 @@
# `mount(component {, options}])`
# `mount(component [, options])`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eddyerburgh
I fixed this format.

@@ -1,4 +1,4 @@
# `shallow(component {, options}])`
# `shallow(component [, options])`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eddyerburgh
I fixed this format.

@38elements
Copy link
Contributor Author

@kazupon
レビューありがとうございます。
指摘していただいた点を修正しました。
よろしくお願いします。

@38elements 38elements closed this Apr 26, 2018
@kazupon
Copy link
Member

kazupon commented Apr 26, 2018

すいません。見切れてなくて。🙇
帰ったら見ますね。

@38elements 38elements reopened this Apr 26, 2018
@38elements
Copy link
Contributor Author

@kazupon
気になさらないでください。

@eddyerburgh eddyerburgh merged commit 498c2fd into vuejs:dev Apr 26, 2018
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