Skip to content

docs: Translate | Essentials > Conditional Rendering #86

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 1 commit into from
Sep 23, 2020

Conversation

noliaki
Copy link
Contributor

@noliaki noliaki commented Sep 23, 2020

resolve #7

Vue2の文章とほぼ変わらなかったので、ほとんどVue2の文章を引用しています。

Vue2の翻訳からの変更点

↓の部分で テンプレート ではなく、<template>としています。
lang-ja...noliaki:translate-conditional_rendering#diff-05d26db877d00579eaa2b28cb102443fR18

Vue2のドキュメント: https://jp.vuejs.org/v2/guide/conditional.html#%E3%83%86%E3%83%B3%E3%83%97%E3%83%AC%E3%83%BC%E3%83%88%E3%81%A7%E3%81%AE-v-if-%E3%81%AB%E3%82%88%E3%82%8B%E6%9D%A1%E4%BB%B6%E3%82%B0%E3%83%AB%E3%83%BC%E3%83%97

疑問点

v-if with v-for の部分で、以下のドキュメントでは v-forv-ifよりも優先度が高いとの旨が記載されていると思うのですが、

以下のマイグレーションガイドだと、 v-ifは常にv-forよりも優先されるといった旨が記載されています。

これはリスト1つ1つに対してのことではなく、例えば以下のようなリスト全体を非表示にしたい場合に v-ifv-forよりも先に評価され、リストそのものをレンダリングしない、という意味でしょうか?

<ul>
  <li v-if="false" v-for="item in items"></li>
</ul>

すみません、本来であればVue3のソースコードを追えば良いものの、追えきれずでして…

レビューお願いいたします🙇‍♂️

@netlify
Copy link

netlify bot commented Sep 23, 2020

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit daa02e8

https://deploy-preview-86--vuejs-v3-ja-doc-preview.netlify.app

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.

レビュー 👀 しました。
翻訳はLGTMです! 🙏

@kazupon
Copy link
Member

kazupon commented Sep 23, 2020

疑問点

v-if with v-for の部分で、以下のドキュメントでは v-forv-ifよりも優先度が高いとの旨が記載されていると思うのですが、

以下のマイグレーションガイドだと、 v-ifは常にv-forよりも優先されるといった旨が記載されています。

これはリスト1つ1つに対してのことではなく、例えば以下のようなリスト全体を非表示にしたい場合に v-ifv-forよりも先に評価され、リストそのものをレンダリングしない、という意味でしょうか?

<ul>
  <li v-if="false" v-for="item in items"></li>
</ul>

すみません、本来であればVue3のソースコードを追えば良いものの、追えきれずでして…

レビューお願いいたします🙇‍♂️

そうですね。v-ifv-for より優先されます。
最近、 Evan 氏がさりげなく、ドキュメントを更新したようです。
vuejs/docs@8235a0d#diff-6530de18a3e51208729971dec82fcd03R1-R16

これ、本家の方、ドキュメント整合性取る必要がありますね。
ひとまず、確認してみます。

@kazupon
Copy link
Member

kazupon commented Sep 23, 2020

issue 上がっていますね。
vuejs/docs#534

@kazupon
Copy link
Member

kazupon commented Sep 23, 2020

結構、いろんなところにあるんですね... 影響範囲大きい 😅

@kazupon
Copy link
Member

kazupon commented Sep 23, 2020

コアチームのドキュメントチームでも把握しているので、後で更新すと思うので、今は一旦現時点での内容にしておきたいと思います。

一旦マージしますね!
翻訳ありがとうございました!

@kazupon kazupon merged commit 938d443 into vuejs-jp:lang-ja Sep 23, 2020
@noliaki noliaki deleted the translate-conditional_rendering branch September 24, 2020 00:38
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.

Essentials > Conditional Rendering の翻訳
3 participants