Skip to content

Conversation

@jay-es
Copy link
Member

@jay-es jay-es commented Sep 25, 2022

resolve #69

内容は前回 PR の Composition API のページとほぼ同じです。
よろしくお願いします!

- すべての同期的な子コンポーネントがマウントされた(非同期コンポーネントや `<Suspense>` ツリー内のコンポーネントは含まれません)。

- Its own DOM tree has been created and inserted into the parent container. Note it only guarantees that the component's DOM tree is in-document if the application's root container is also in-document.
- そのコンポーネント自身の DOM ツリーが作成され、親コンテナーに挿入された。アプリケーションのルートコンテナが document 内にある場合のみ、そのコンポーネントの DOM ツリーも document 内にあることを保証することに注意してください。
Copy link
Member

Choose a reason for hiding this comment

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

挿入された。 で終わるの不自然なので以下のようにすると自然が文章になりそうです。

Suggested change
- そのコンポーネント自身の DOM ツリーが作成され、親コンテナーに挿入された。アプリケーションのルートコンテナが document 内にある場合のみ、そのコンポーネントの DOM ツリーも document 内にあることを保証することに注意してください。
- そのコンポーネント自身の DOM ツリーが作成され、親コンテナーに挿入されます。アプリケーションのルートコンテナが document 内にある場合のみ、そのコンポーネントの DOM ツリーも document 内にあることを保証することに注意してください。

Copy link
Member Author

Choose a reason for hiding this comment

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

こちらは「された時」に修正しました。
(関連箇所や composition-api-lifecycle.md も同様に修正)

- Custom directive hooks
- Transition hooks
- コンポーネントのレンダリング
- イベント ハンドラー
Copy link
Member

Choose a reason for hiding this comment

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

ここは空白入れる必要なさそうです。

Suggested change
- イベント ハンドラー
- イベントハンドラー

Copy link
Member Author

Choose a reason for hiding this comment

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

下の行の「ライフサイクル フック」と、composition-api-lifecycle.md の同じ箇所も修正しました

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.

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

@jay-es jay-es requested a review from kazupon October 2, 2022 11:06
@kazupon
Copy link
Member

kazupon commented Oct 2, 2022

いつもありがとうございます! 💚
LGTMです!

@kazupon kazupon merged commit 59399fd into vuejs-translations:main Oct 2, 2022
@jay-es jay-es deleted the 69-lifecycle branch October 2, 2022 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

「API > Options API > Options: Lifecycle」の翻訳

2 participants