Skip to content

Translate: Migration Guide > KeyCode Modifiers #156

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 2 commits into from
Nov 10, 2020

Conversation

shimadev
Copy link
Contributor

@shimadev shimadev commented Nov 2, 2020

Description of Problem

resolve #121

Proposed Solution

Additional Information

https://github.com/vuejs-jp/ja.vuejs.org/blob/lang-ja/src/guide/migration/keycode-modifiers.md
こちらの翻訳です
遅くなってすみません。レビューよろしくお願いします!

@netlify
Copy link

netlify bot commented Nov 2, 2020

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

Built with commit aa55900

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

Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

ありがとうございます!
いくつかコメントしました


For those using `keyCode` in their codebase, we recommend converting them to their kebab-cased named equivalents.
キーコードを利用している場合は、ケバブケース名を利用する方法に変更することを推奨します。
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
キーコードを利用している場合は、ケバブケース名を利用する方法に変更することを推奨します
キーコードを利用している場合は、ケバブケースでの命名に変更することを推奨します

日本語として少し違和感があるので、 naming を 命名 と訳した上で方法というのをそこに含めてしまうほうが読みやすいかなと!

<input v-on:keyup.f1="showHelpText" />
```

## 3.x Syntax
## 3.x 構文
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
## 3.x 構文
## 3.x での構文

ここ申し訳ないのですが、 での構文 で統一させてほしいです!


## 2.x Syntax
## 2.x 構文
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
## 2.x 構文
## 2.x での構文

ここ申し訳ないのですが、 での構文 で統一させてほしいです!

@shimadev
Copy link
Contributor Author

shimadev commented Nov 4, 2020

レビューありがとうございます!
修正しました!

Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

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

@potato4d potato4d merged commit 175961c into vuejs-jp:lang-ja Nov 10, 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.

Migration Guide > KeyCode Modifiers の翻訳
2 participants