Skip to content

docs: Essentials > Components Basics の翻訳 #11 #98

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 10 commits into from
Oct 1, 2020

Conversation

kou1127h
Copy link
Contributor

Description of Problem

resolve #11

Additional Information

https://jp.vuejs.org/v2/guide/components.html
と重複する内容が多かったので、基本的にはこちらを踏襲しています。

@netlify
Copy link

netlify bot commented Sep 29, 2020

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

Built with commit c09b423

https://deploy-preview-98--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.

レビュー 👀 コメント 💬 しました。
翻訳内容については、大丈夫そうです。
細かい部分、コメントしましたが、対応お願いします。 🙏

@@ -215,7 +215,9 @@ app.component('blog-post', {
問題は、このボタンが何もしないことです:

```html
<button>Enlarge text</button>
<button>
Copy link
Member

Choose a reason for hiding this comment

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

textlint実行していただいてありがとうございます!
codeのところ、今の設定だと反応してしまうんですね。
なるべく、今後のドキュメントのメンテ上、行構成などあまり構造を変えたくないので、lintで対応したところ、こちら一旦原文と同じく、戻していただけますでしょうか? 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

お返事遅れてすみません!

一旦原文と同じく、戻していただけますでしょうか?

こちらの修正は一度僕の環境で誤ってフォーマットされたものを原文準拠に戻したものになっております。
ので、この辺りに関して修正は不要な認識ですがいかがでしょうか?

Copy link
Member

Choose a reason for hiding this comment

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

ありがとうございます。
そうですね。
このままで大丈夫です。

@@ -215,7 +215,9 @@ app.component('blog-post', {
問題は、このボタンが何もしないことです:

```html
<button>Enlarge text</button>
<button>
Copy link
Member

Choose a reason for hiding this comment

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

ありがとうございます。
そうですね。
このままで大丈夫です。

@kazupon
Copy link
Member

kazupon commented Oct 1, 2020

対応ありがとうございました。
CIでエラーが起きていますが、翻訳作業に支障はない(このPRとは別事象)なので、マージしますね!

翻訳ありがとうございました!

@kazupon kazupon merged commit e2c21e0 into vuejs-jp:lang-ja Oct 1, 2020
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 > Components Basics の翻訳
2 participants