Skip to content

docs: Reusability & Composition > Mixins の翻訳 #83

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 2 commits into from
Sep 24, 2020

Conversation

Nag729
Copy link
Collaborator

@Nag729 Nag729 commented Sep 22, 2020

resolve #25
Reusability & Composition > Mixins の翻訳

レビューよろしくお願いします!

@netlify
Copy link

netlify bot commented Sep 22, 2020

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit b145886

https://deploy-preview-83--vuejs-v3-ja-doc-preview.netlify.app

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.

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


To address these issues, we added a new way to organize code by logical concerns: the [Composition API](composition-api-introduction.html).
これらの問題に対処するため、論理的な関心事によってコードを整理する新しい方法を追加しました: [Composition API](composition-api-introduction.html)
Copy link
Member

Choose a reason for hiding this comment

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

Composition API の訳は以下のように統一させてください。:pray:

Suggested change
これらの問題に対処するため、論理的な関心事によってコードを整理する新しい方法を追加しました: [Composition API](composition-api-introduction.html)
これらの問題に対処するため、論理的な関心事によってコードを整理する新しい方法を追加しました: [コンポジション API](composition-api-introduction.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.

Suggested change に沿って修正を行いました。

@@ -196,7 +196,7 @@ app.mixin({
})
```

As you can see, in the console we have `toVal` and `fromVal` printed first from the mixin and then from the `app`. We always return `fromVal` if it exists, that's why `this.$options.custom` is set to `hello!` in the end. Let's try to change a strategy to _always return a value from the child instance_:
ご覧の通り、コンソールには最初にミックスインから、次に `app` から出力された `toVal` `fromVal` が表示されます。`fromVal` が存在する場合には常にそれが返されるため、最終的には `this.$options.custom` `hello!` がセットされます。ストラテジを*常に子インスタンスの値を返す*ように変更してみましょう:
Copy link
Member

Choose a reason for hiding this comment

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

ご覧の通り は、これまでの Vue 公式ドキュメントの翻訳トーンに合わないので以下の意訳でお願いします!

Suggested change
ご覧の通り、コンソールには最初にミックスインから、次に `app` から出力された `toVal``fromVal` が表示されます。`fromVal` が存在する場合には常にそれが返されるため、最終的には `this.$options.custom``hello!` がセットされます。ストラテジを*常に子インスタンスの値を返す*ように変更してみましょう:
上記の通り、コンソールには最初にミックスインから、次に `app` から出力された `toVal``fromVal` が表示されます。`fromVal` が存在する場合には常にそれが返されるため、最終的には `this.$options.custom``hello!` がセットされます。ストラテジを*常に子インスタンスの値を返す*ように変更してみましょう:

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change に沿って修正を行いました。

@Nag729
Copy link
Collaborator Author

Nag729 commented Sep 24, 2020

レビューいただいた指摘に対応しました!
ご確認よろしくお願いします🙏

@kazupon
Copy link
Member

kazupon commented Sep 24, 2020

確認しました。
LGTMです!
CIでエラーが起きていますが、翻訳作業に支障がない(このPRとは別事象)ので、マージします!

翻訳ありがとうございました!

@kazupon kazupon merged commit 2775e20 into vuejs-jp:lang-ja Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reusability & Composition > Mixins の翻訳
2 participants