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

Guide > Enter & Leave Transitions の翻訳を追従 #307

Merged

Conversation

Katakuranatsumi
Copy link
Contributor

@Katakuranatsumi Katakuranatsumi commented May 3, 2021

Description of Problem

Guide > Enter & Leave Transitions の翻訳を追従しました。
https://v3.vuejs.org/guide/transitions-enterleave.html

Proposed Solution

ファイルの更新履歴
https://github.com/vuejs/docs-next/commits/master/src/guide/transitions-enterleave.md

前回からの差分
vuejs/docs@5d2a1e3#diff-bd22f27bea3e777dd5f5f0393cc3bca3482956bf9a395876b15aefd64cdcbf65
vuejs/docs@40e361f#diff-bd22f27bea3e777dd5f5f0393cc3bca3482956bf9a395876b15aefd64cdcbf65
vuejs/docs@3f699df#diff-bd22f27bea3e777dd5f5f0393cc3bca3482956bf9a395876b15aefd64cdcbf65

(vuejs/docs@722b92e#diff-bd22f27bea3e777dd5f5f0393cc3bca3482956bf9a395876b15aefd64cdcbf65 の差分は、すでにGuide > Introduction の翻訳を追従で対応していただいていたため対応しておりませんmm)

Additional Information

翻訳の追従の他にも、原文と行のずれがあった箇所を以下のように修正しています🙏

  • 73行目〜83行目までの番号付きリストの間に、原文にならって空白の行を追加
  • 295行目と315行目のコメントを、原文にならって1行から2行にするよう修正

@netlify
Copy link

netlify bot commented May 3, 2021

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

Built with commit 88c405d

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

@Katakuranatsumi Katakuranatsumi changed the title 【WIP】Guide > Enter & Leave Transitions の翻訳を追従 Guide > Enter & Leave Transitions の翻訳を追従 May 3, 2021
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.

@@ -189,7 +189,7 @@ Vue.createApp(Demo).mount('#demo')
     transform: scale(0);
   }
   50% {
-    transform: scale(1.5);
+    transform: scale(1.25);
   }
   100% {
     transform: scale(1);

この差分が抜けちゃってるかもしれないです。

<component :is="view"></component>
</transition>
<div id="demo">
<input v-model="view" type="radio" value="v-a" id="a"><label for="a">A</label> TODO: update to Vue 3
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
<input v-model="view" type="radio" value="v-a" id="a"><label for="a">A</label> TODO: update to Vue 3
<input v-model="view" type="radio" value="v-a" id="a"><label for="a">A</label>

L506 で消したものが混入しちゃったかもしれませんね。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ご確認ありがとうございましたmm
失礼しました、 5b3da7c で削除しました🙇‍♀️

@@ -7,7 +7,7 @@ Vue は、DOM からアイテムが追加、更新、削除されたときにト
- トランジションフックが実行されている間、JavaScript を使って直接 DOM 操作を行います。
- サードパーティの JavaScript アニメーションライブラリと連携します。

このページでは、entering/leaving とリストのトランジションについてのみ取り扱いますが、次の章では、[状態のトランジション](transitions-state.html) について扱います。
このページでは、entering/leaving とリストのトランジションについてのみ取り扱いますが、次の章では、[リストのトランジション](transitions-list.html) と [状態のトランジション](transitions-state.html) について扱います。
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
このページでは、entering/leaving とリストのトランジションについてのみ取り扱いますが、次の章では、[リストのトランジション](transitions-list.html)[状態のトランジション](transitions-state.html) について扱います。
このページでは、entering/leaving のトランジションについてのみ取り扱いますが、次の章では、[リストのトランジション](transitions-list.html)[状態のトランジション](transitions-state.html) について扱います。

リストのトランジションは別ページにという説明に変わったんですね。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

「リストのトランジション」のままになっていました、失礼しました🙇‍♀️
88c405d で修正しましたmm

@Katakuranatsumi
Copy link
Contributor Author

Katakuranatsumi commented May 4, 2021

レビューありがとうございますmm
5f8e6fe でL182が transform: scale(1.25); になるよう修正しました🙏

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.

ありがとうございました 👍🏼
LGTM!!

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