Skip to content

Document gotcha with :is and DOM props #1955

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 5 commits into from
Dec 10, 2019

Conversation

inosy22
Copy link
Contributor

@inosy22 inosy22 commented Dec 5, 2019

概要

resolve #1947

補足

こちらのコミットの翻訳になります。
vuejs/v2.vuejs.org@3d96b74

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

Comment on lines 607 to 609
この属性は通常の HTML 要素にも使えますが、コンポーネントとして取り扱われることに注意してください。
つまり、全ての属性が ** DOM の属性としてバインドされる** ということになります。
`value` などの一部のプロパティが、期待通りに動作するためには、[`.prop` 修飾子](../api/#v-bind) を使ってバインドする必要があります。
Copy link
Collaborator

Choose a reason for hiding this comment

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

nits: 細かいですが、原文では1段落なので合わせていただいてもよいでしょうか?

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

レビューありがとうございます!後ほど対応いたします!

Copy link
Collaborator

@oohira oohira left a comment

Choose a reason for hiding this comment

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

@inosy22 翻訳ありがとうございます!スタイルの調整だけお願いします 🙏

@inosy22
Copy link
Contributor Author

inosy22 commented Dec 10, 2019

@oohira 修正させていただきました!
お手数ですが再度ご確認いただけると幸いです。

@oohira oohira merged commit d21255c into vuejs:lang-ja Dec 10, 2019
@oohira
Copy link
Collaborator

oohira commented Dec 10, 2019

@inosy22 LGTM. ありがとうございました! 👍

@vue-bot
Copy link
Contributor

vue-bot commented Dec 10, 2019

Hey @inosy22, thank you for your time and effort spent on this PR, contributions like yours help make Vue better for everyone. Cheers! 💚

@inosy22
Copy link
Contributor Author

inosy22 commented Dec 12, 2019

ご確認ありがとうございました!

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.

[Doc]: Document gotcha with :is and DOM props
3 participants