Skip to content
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

Fix on guide/composition-api-setup.md #374

Merged
merged 1 commit into from
Jun 4, 2021

Conversation

nesso-pfl
Copy link
Contributor

@nesso-pfl nesso-pfl commented Jun 3, 2021

こんにちは✋ このドキュメントは VueJS の勉強にとても良く、熱心に読ませていただいています。
今回は和訳の改善提案をさせてください。

Description of Problem

現状、以下の英文(ソース)に対し:

If title is an optional prop, it could be missing from props. In that case, toRefs won't create a ref for title. Instead you'd need to use toRef:

以下の和訳(ソース)が割り当てられています:

title オプションのプロパティである場合、 props から抜けている可能性があります。その場合、 toRefs では title の ref はつくられません。代わりに toRef を使う必要があります:

このドキュメントでは、 setupdata のような API を「オプション」と呼ぶことがあるのも相まって、「オプション」という訳だと理解の妨げになるように感じました。

Proposed Solution

optional を「省略可能な」と和訳することを提案させてください。
或いは単純に「オプショナルな」という訳も良いかもしれません。

もし、「オプション」の訳のままにしたい場合でも「 title がオプションのプロパティである場合...」という具合に助詞があるべきだと考えています。

Additional Information

@netlify
Copy link

netlify bot commented Jun 3, 2021

✔️ Deploy Preview for vuejs-v3-ja-doc-preview ready!

🔨 Explore the source changes: 4304322

🔍 Inspect the deploy log: https://app.netlify.com/sites/vuejs-v3-ja-doc-preview/deploys/60b9419f86b135000736ec9a

😎 Browse the preview: https://deploy-preview-374--vuejs-v3-ja-doc-preview.netlify.app

Copy link
Member

@naokie naokie left a comment

Choose a reason for hiding this comment

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

ありがとうございます!
参照先まで提示いただいて、わかりやすかったです。

個人的にも API まわりのドキュメントを翻訳中に同じようなことを考えていました。

@naokie naokie merged commit 44ce407 into vuejs-jp:lang-ja Jun 4, 2021
naokie added a commit to naokie/ja.vuejs.org that referenced this pull request Jun 4, 2021
@nesso-pfl nesso-pfl deleted the fix-on-composition-api-setup branch June 4, 2021 13:13
@naokie naokie mentioned this pull request Jun 5, 2021
17 tasks
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.

2 participants