-
Notifications
You must be signed in to change notification settings - Fork 87
docs: Components In-Depth > Dynamic & Async Components の翻訳 #90
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
docs: Components In-Depth > Dynamic & Async Components の翻訳 #90
Conversation
Deploy preview for vuejs-v3-ja-doc-preview ready! Built with commit a9ad2b8 https://deploy-preview-90--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.
レビュー 👀 コメント 💬 しました。
翻訳内容については、大丈夫です。
細かいところでコメントしましたので、確認お願いします!
|
||
Async components are _suspensible_ by default. This means if it has a [`<Suspense>`](TODO) in the parent chain, it will be treated as an async dependency of that `<Suspense>`. In this case, the loading state will be controlled by the `<Suspense>`, and the component's own loading, error, delay and timeout options will be ignored. | ||
非同期コンポーネントはデフォルトで _suspensible_ です。これは非同期コンポーネントが [`<Suspense>`](TODO) を親に持ったとき、 `<Suspense>` の非同期の依存として取り扱われることを意味しています。このケースでは、読み込みの状態は `<Suspense>` から制御され、コンポーネント自身が持つ loading や error, delay, timeout といったオプションは無視されます。 |
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.
以下、個人的メモです。
なんだろう。この TODOは... 🤔 執筆時に suspense は正式機能としてはリリースできるかどうか分からないから、メモとして残しておいた感じかな? チームに確認してみる。
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.
自分も翻訳するときにギョッとしました。Suspenseのページが存在しないのにここだけSuspenseについて取り扱っていて、唐突感がすごいですよねw
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.
そうですねw
Suspence 、高度な内容なので個人的には、別セクションに分けて欲しいなと思っています。
その辺含めてフィードバックしておきますね!
Co-authored-by: kazuya kawaguchi <kawakazu80@gmail.com>
Co-authored-by: kazuya kawaguchi <kawakazu80@gmail.com>
Co-authored-by: kazuya kawaguchi <kawakazu80@gmail.com>
@kazupon レビューありがとうございます!Suggestion 頂いたところは取り込みました。 |
確認しました! CIでエラーが起きていますが、翻訳作業に支障はない(このPRとは別事象)なので、マージしますね。 |
Description of Problem
resolve #18
https://deploy-preview-90--vuejs-v3-ja-doc-preview.netlify.app/guide/component-dynamic-async.html で動作確認できます。
Proposed Solution
Additional Information