-
Notifications
You must be signed in to change notification settings - Fork 87
Guide > Components In-Depth > Slots の翻訳を追従 #293
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
@@ -222,7 +221,8 @@ Vue には [Web Components spec draft](https://github.com/w3c/webcomponents/blob | |||
</footer> | |||
</div> | |||
``` | |||
**`v-slot` は([一つの例外](#デフォルトスロットしかない場合の省略記法) を除き) `<template>` にしか指定できないことに注意してください。 | |||
|
|||
**`v-slot` は([一つの例外](#デフォルトスロットしかない場合の省略記法) を除き) `<template>` にしか指定できないことに注意してください。 |
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.
「デフォルトスロット」の「デ」に差分があってページ内リンクが動作してなかったため修正しました。
Deploy preview for vuejs-v3-ja-doc-preview ready! Built with commit f091edc https://deploy-preview-293--vuejs-v3-ja-doc-preview.netlify.app |
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.
レビューしました。
確認お願いします!
@@ -247,8 +247,7 @@ app.component('todo-list', { | |||
}) | |||
``` | |||
|
|||
親コンポーネントでこれをカスタマイズするために、スロットに交換してもよいでしょう: | |||
|
|||
親コンポーネントでこれをカスタマイズするために、<span v-pre>`{{ item }}`</span> を `<slot>` に置き換えたい場合があります: |
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.
原文ですが、
We might want to replace the {{ item }} with a <slot> to customize it on parent component:
となっていて、<span v-pre>
と </span>
はいらないと思います。
https://v3.vuejs.org/guide/component-slots.html#scoped-slots
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.
ここで問題になっていた下図の現象、日本語訳でも起きちゃうんですよね。
原文はこちらです。
https://github.com/vuejs/docs-next/blob/master/src/guide/component-slots.md#L250
VuePress の hack のようなものなのですが、そのまま取り込もうと思います。
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.
原文と同じく、取り込みましょう!
レビューいただいた箇所、コメントお返ししたので再レビューお願いします。 |
Description of Problem
Guide > Components In-Depth > Slots の翻訳を追従しました。
https://v3.vuejs.org/guide/component-slots.html
Proposed Solution
ファイル変更履歴
https://github.com/vuejs/docs-next/commits/master/src/guide/component-slots.md
前回からの差分
vuejs/docs@fc1449b#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
vuejs/docs@b78e978#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
vuejs/docs@b6070fe#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
vuejs/docs@a2f811f#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
vuejs/docs@705fea0#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
vuejs/docs@aefafd4#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
vuejs/docs@4a333dd#diff-35274a894b839664d9b049b61f4ceb59208a220f0008127c91fd80249d07a0ec
Additional Information
原文と行が一致していないところを修正しました。
#94 にしたがって「バインド -> 束縛」に変更しました。