-
Notifications
You must be signed in to change notification settings - Fork 87
Migration Guide > Functional Components の翻訳 #142
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
Deploy preview for vuejs-v3-ja-doc-preview ready! Built with commit fc80c14 https://deploy-preview-142--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.
概ね良さそうな気がします。Using の取り扱いについてコメントしました 📝
|
||
Using the `<dynamic-heading>` component, which is responsible for rendering out the appropriate heading (i.e., `h1`, `h2`, `h3`, etc.), this could have been written as a single-file component in 2.x as: | ||
適切な見出し(つまり `h1`, `h2`, `h3` など)のレンダリングを担当する `<dynamic-heading>` コンポーネントは、2.x では単一ファイルコンポーネントとして次のように記述できました: |
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.
コメントにあった部分ここですよね。
<dynamic-heading>
コンポーネントを使用すると〜から初めて先に Using the <dynamic-heading>
component を消化し、which is ~ を読点のあとに持っていくのが読みやすいかもしれないですね
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.
L92 が使うと〜なので、同じように使うと〜で統一するほうが良さそうだろうか
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.
@potato4d レビューありがとうございます!
提案いただいた内容ですと、以下のように直すイメージでよろしいでしょうか?
<dynamic-heading>
コンポーネントを使うと、適切な見出し(つまりh1
,h2
,h3
など)のレンダリングを担当するコンポーネントは 2.x では単一ファイルコンポーネントとして次のように記述できました:
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.
@potato4d 修正しましたのでご確認よろしくお願いします:pray:
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.
LGTM!
resolve #116
https://v3.vuejs.org/guide/migration/functional-components.html の翻訳です。
なるべく「一語一句翻訳」のガイドラインに沿って翻訳しております。
32行目について、冒頭の
Using
のうまい訳が思いつかず、軽い意訳になっております。118行目のリンクはページの翻訳が終わってからタイトルを合わせる必要がありそうです。(取り急ぎ、ガイドのタイトルに合わせてあります)
ご確認よろしくお願いします。