-
Notifications
You must be signed in to change notification settings - Fork 665
[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
Conversation
This is great @sunecosuri using-with-vuex.md will be rewritten soon, so you should wait until it's been rewritten before translating. |
@sunecosuri wow! thank you for your translation |
563387d
to
ac3acc6
Compare
38f9e94
to
2d999e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
まずは、APIの部分レビュー 👀 しました。
コメント 💬 の部分対応お願いします。
レビューして気がついたのですが、最近のコミット内容が取り込まれていないようなので、翻訳の方にもsyncしておいてください。 🙏
docs/ja/api/mount.md
Outdated
|
||
- **引数:** | ||
|
||
- `{Component} コンポーネント(component)` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
各引数ですが、この辺は、コード書いているときに実際に指定する引数名なので、特に訳さなくても大丈夫ですね。
docs/ja/api/options.md
Outdated
@@ -0,0 +1,165 @@ | |||
# マウンティングオプション | |||
|
|||
`mount`と`shallow`オプション。optionsオブジェクトには、`vue-test-utils`のマウントオプションと生のVueオプションの両方を含めることができます。 |
There was a problem hiding this comment.
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
の訳も入れておいた方がわかりやすいと思います。
docs/ja/api/options.md
Outdated
|
||
`mount`と`shallow`オプション。optionsオブジェクトには、`vue-test-utils`のマウントオプションと生のVueオプションの両方を含めることができます。 | ||
|
||
新しくインスタンスが作成されると、Vueオプションがコンポーネントに渡されます。例:`store`、`propsData`など。完全なリストについては[Vue API docs](https://vuejs.org/v2/api/)を参照してください。 |
There was a problem hiding this comment.
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/)
のように公式日本語サイトにしておくと、ユーザーフレンドリですね。 😉
docs/ja/api/options.md
Outdated
|
||
- 型: `Object` | ||
|
||
コンテキストを機能コンポーネントに渡します。 機能コンポーネントのみで使用できます。 |
There was a problem hiding this comment.
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#関数型コンポーネント
docs/ja/api/options.md
Outdated
|
||
- 型: `Vue` | ||
|
||
コンポーネントのマウント時に使用する[createLocalVue](./createLocalVue.md)によって作成されたVueのローカルコピーです。 |
There was a problem hiding this comment.
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.
の訳が抜けているのでお願いします! 🙏
docs/ja/api/wrapper-array/update.md
Outdated
|
||
`WrapperArray`の各`Wrapper`のルートVueコンポーネントを強制的に再レンダリングします。 | ||
|
||
VueコンポーネントをWrapperArrayで呼び出した場合、各Vueコンポーネントは強制的に再レンダリングされます。 |
There was a problem hiding this comment.
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コンポーネント配列において呼び出された場合
が分かりやすくないでしょうか?
docs/ja/api/wrapper/README.md
Outdated
@@ -0,0 +1,16 @@ | |||
# ラッパー |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vue.js 全般の翻訳では、なるべく、コンピュータ用語は伸ばさないよう統一するようにしているのでここでは、ラッパ
にしましょう!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
他の部分もありましたら、対応お願いします! 🙏
docs/ja/api/wrapper/README.md
Outdated
|
||
- **プロパティ:** | ||
|
||
`vm` `Component`:これはvueのインスタンスです。`wrapper.vm`を使って[vmのプロパティとインスタンスメソッド](https://vuejs.org/v2/api/#Instance-Properties)にアクセスできます。これは、Vueコンポーネントラッパーにのみ存在します。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここのリンク先も日本語公式サイトのドキュメントになっているとユーザーフレンドリですね! 😉
docs/ja/api/wrapper/contains.md
Outdated
# contains(selector) | ||
|
||
- **引数:** | ||
- `{string|Component} セレクタ(selector)` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
引数の部分は訳さなくでも大丈夫 🙆 です!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
他の部分でも訳されている場合はお願いします!
docs/ja/api/wrapper/hasAttribute.md
Outdated
expect(wrapper.hasAttribute('id', 'foo')).to.equal(true) | ||
``` | ||
|
||
- **その他:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternative
は他の選択肢という意味だと思うので、代替
とかいかがでしょうか?
@kazupon まだざっくりしか訳してなかったので沢山ご指摘いただいて申し訳ないです。明日までに最新のコミット取り込んで反映しておきます! |
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 |
@eddyerburgh |
@kazupon 一通りレビューいただいた分終了いたしましたので、追加分をレビューよろしくお願い致します! |
@sunecosuri ちょっとレビュー指摘内容箇所が多いので、こちらで対応しました。 @eddyerburgh /cc @yyx990803 |
@kazupon 承知しました。引き続きよろしくお願いします。 |
@sunecosuri こちらで、差分を取り込んでまたPRしますね。 |
TODO