Skip to content

[WIP]Translate docs into ja #58

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

Closed
wants to merge 53 commits into from
Closed

Conversation

sunecosuri
Copy link
Contributor

@sunecosuri sunecosuri commented Sep 30, 2017

TODO

  • README.md
  • SUMMARY.md
  • api
    • README.md
    • createLocalVue.md
    • mount.md
    • options.md
    • selectors.md
    • shallow.md
    • wrapper-array
      • README.md
      • at.md
      • contains.md
      • hasAttribute.md
      • hasClass.md
      • hasProp.md
      • hasStyle.md
      • is.md
      • isEmpty.md
      • isVueInstance.md
      • setData.md
      • setMethods.md
      • setProps.md
      • trigger.md
      • update.md
    • wrapper
      • README.md
      • contains.md
      • emmited.md
      • emmitedByOrder.md
      • exists.md
      • find.md
      • findAll.md
      • hasAttribute.md
      • hasClass.md
      • hasProp.md
      • hasStyle.md
      • html.md
      • is.md
      • isEmpty.md
      • isVueInstance.md
      • name.md
      • setData.md
      • setMethods.md
      • setProps.md
      • text.md
      • trigger.md
      • update.md
  • guides
    • choosing-a-test-runner.md
    • common-tips.md
    • getting-started.md
    • README.md
    • testing-SFCs-with-jest.md
    • testing-SFCs-with-mocha-webpack.md
    • using-with-vuex.md(pending)

@eddyerburgh
Copy link
Member

This is great @sunecosuri

using-with-vuex.md will be rewritten soon, so you should wait until it's been rewritten before translating.

@kazupon
Copy link
Member

kazupon commented Oct 2, 2017

@sunecosuri wow! thank you for your translation
I will review it later! 😉

@sunecosuri sunecosuri force-pushed the translate-ja branch 2 times, most recently from 563387d to ac3acc6 Compare October 5, 2017 01:07
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.

まずは、APIの部分レビュー 👀 しました。
コメント 💬 の部分対応お願いします。

レビューして気がついたのですが、最近のコミット内容が取り込まれていないようなので、翻訳の方にもsyncしておいてください。 🙏


- **引数:**

- `{Component} コンポーネント(component)`
Copy link
Member

Choose a reason for hiding this comment

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

各引数ですが、この辺は、コード書いているときに実際に指定する引数名なので、特に訳さなくても大丈夫ですね。

@@ -0,0 +1,165 @@
# マウンティングオプション

`mount`と`shallow`オプション。optionsオブジェクトには、`vue-test-utils`のマウントオプションと生のVueオプションの両方を含めることができます。
Copy link
Member

Choose a reason for hiding this comment

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

Options for mount and shallow. ですが、 mount と shallow に対するオプション。for の訳も入れておいた方がわかりやすいと思います。


`mount`と`shallow`オプション。optionsオブジェクトには、`vue-test-utils`のマウントオプションと生のVueオプションの両方を含めることができます。

新しくインスタンスが作成されると、Vueオプションがコンポーネントに渡されます。例:`store`、`propsData`など。完全なリストについては[Vue API docs](https://vuejs.org/v2/api/)を参照してください。
Copy link
Member

Choose a reason for hiding this comment

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

Vue API docs のリンクですが、[Vue API ドキュメント](https://jp.vuejs.org/v2/api/)のように公式日本語サイトにしておくと、ユーザーフレンドリですね。 😉


- 型: `Object`

コンテキストを機能コンポーネントに渡します。 機能コンポーネントのみで使用できます。
Copy link
Member

Choose a reason for hiding this comment

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

functional component の訳ですが、公式サイトでは、関数型コンポーネントにしているので合わせておいてください。:pray:
https://jp.vuejs.org/v2/guide/render-function.html#関数型コンポーネント


- 型: `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.

Installing plugins on this copy of Vue prevents polluting the original Vue copy. の訳が抜けているのでお願いします! 🙏


`WrapperArray`の各`Wrapper`のルートVueコンポーネントを強制的に再レン​​ダリングします。

VueコンポーネントをWrapperArrayで呼び出した場合、各Vueコンポーネントは強制的に再レン​​ダリングされます。
Copy link
Member

Choose a reason for hiding this comment

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

If called on a Vue component wrapper array,の訳ですが、ラップされたVueコンポーネント配列において呼び出された場合が分かりやすくないでしょうか?

@@ -0,0 +1,16 @@
# ラッパー
Copy link
Member

Choose a reason for hiding this comment

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

Vue.js 全般の翻訳では、なるべく、コンピュータ用語は伸ばさないよう統一するようにしているのでここでは、ラッパにしましょう!

Copy link
Member

Choose a reason for hiding this comment

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

他の部分もありましたら、対応お願いします! 🙏


- **プロパティ:**

`vm` `Component`:これはvueのインスタンスです。`wrapper.vm`を使って[vmのプロパティとインスタンスメソッド](https://vuejs.org/v2/api/#Instance-Properties)にアクセスできます。これは、Vueコンポーネントラッパーにのみ存在します。
Copy link
Member

Choose a reason for hiding this comment

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

ここのリンク先も日本語公式サイトのドキュメントになっているとユーザーフレンドリですね! 😉

# contains(selector)

- **引数:**
- `{string|Component} セレクタ(selector)`
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
Member

Choose a reason for hiding this comment

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

他の部分でも訳されている場合はお願いします!

expect(wrapper.hasAttribute('id', 'foo')).to.equal(true)
```

- **その他:**
Copy link
Member

Choose a reason for hiding this comment

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

Alternative は他の選択肢という意味だと思うので、代替とかいかがでしょうか?

@sunecosuri
Copy link
Contributor Author

sunecosuri commented Oct 6, 2017

@kazupon まだざっくりしか訳してなかったので沢山ご指摘いただいて申し訳ないです。明日までに最新のコミット取り込んで反映しておきます!

@eddyerburgh
Copy link
Member

I think we should leave out using-with-vuex and merge when the other translations are complete. I'll be rewriting using-with-vuex, but it might take me a couple of weeks to do

@sunecosuri
Copy link
Contributor Author

@eddyerburgh
OK. 'using-with-vuex 'is not included in this PR.
I will translate it again after the rewriting is over

@sunecosuri
Copy link
Contributor Author

@kazupon 一通りレビューいただいた分終了いたしましたので、追加分をレビューよろしくお願い致します!

@kazupon
Copy link
Member

kazupon commented Oct 10, 2017

@sunecosuri ちょっとレビュー指摘内容箇所が多いので、こちらで対応しました。
https://github.com/kazupon/vue-test-utils/tree/translate-ja

@eddyerburgh /cc @yyx990803
could you allow write permission to this repo ?
I would like to push additional commits this PR.

@sunecosuri
Copy link
Contributor Author

sunecosuri commented Oct 10, 2017

@kazupon 承知しました。引き続きよろしくお願いします。
頃合いを見てこのPRも閉じたいと思います。

@sunecosuri sunecosuri closed this Oct 11, 2017
@kazupon
Copy link
Member

kazupon commented Oct 12, 2017

@sunecosuri
すいません、返事が遅くなりました。 🙇
閉じてしまった後にあれですが、これまでの翻訳していただいたcommits、尊重したかったので、
このPRに追加commitができればと思っていました。

こちらで、差分を取り込んでまたPRしますね。
翻訳ありがとうございました!

@kazupon kazupon mentioned this pull request Oct 13, 2017
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