Skip to content

update ja docs #2602

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 11 commits into from
Feb 8, 2019
Merged

update ja docs #2602

merged 11 commits into from
Feb 8, 2019

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented Jan 30, 2019

ping @ktsn
Could you review it please? 🙏

@kazupon kazupon changed the title update docs ja update ja docs Jan 31, 2019
@@ -15,4 +15,4 @@ Vue Router は [Vue.js](http://vuejs.org) 公式ルータです。これは Vue.
- HTML5 history モードまたは hash モードと IE9 の互換性
- カスタマイズ可能なスクロール動作

[はじめに](./guide/) または [例](https://github.com/vuejs/vue-router/tree/dev/examples) から試してください。 ([`README.md`](https://github.com/vuejs/vue-router/)を参照してください。).
[ドキュメント](https://router.vuejs.org/ja/guide/)で始めるか、 [例](https://github.com/vuejs/vue-router/tree/dev/examples) から試してください。 ([`README.md`](https://github.com/vuejs/vue-router/)を参照してください。).
Copy link
Member

Choose a reason for hiding this comment

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

これ前のほうが自然な気がするのですが、変える理由ってなんでしょうか?

Copy link
Member Author

Choose a reason for hiding this comment

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

あ、もう一度、元コミット確認したら、 repository トップの README.md でしたね。。。113537f
revert します。 🙇

path: '*'
}
{
// will match anything starting with `/user-`
// `/user-`から始まるもにマッチします
Copy link
Member

Choose a reason for hiding this comment

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

始まるもの の typo?

path: '*'
}
{
// will match anything starting with `/user-`
// `/user-`から始まるもにマッチします
Copy link
Member

Choose a reason for hiding this comment

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

細かいですが anything の訳も入れたいですね。

path: '/user-*'
}
```

When using _asterisk_ routes, make sure to correctly order your routes so that _asterisk_ ones are at the end.
The route `{ path; '*' }` is usually used to 404 client side. If you are using _History mode_, make sure to [correctly configure your server](./history-mode.md) as well.
_アスタリスク_ ルートを使用するときは、_アスタリスク_ ルートが最後になるようにルートを正しく順序付けでください。
Copy link
Member

Choose a reason for hiding this comment

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

順序付け ?

When using _asterisk_ routes, make sure to correctly order your routes so that _asterisk_ ones are at the end.
The route `{ path; '*' }` is usually used to 404 client side. If you are using _History mode_, make sure to [correctly configure your server](./history-mode.md) as well.
_アスタリスク_ ルートを使用するときは、_アスタリスク_ ルートが最後になるようにルートを正しく順序付けでください。
`{ path: '*' }` ルートは、通常 404 クライアントサイドで使われます。_History モード_ を使用する場合は、[正しいサーバの設定](./history-mode.md)も同様に確認してください。
Copy link
Member

Choose a reason for hiding this comment

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

404 クライアントサイド

ちょっと分かりづらいので、「クライアントサイドの 404 ページ」みたいな感じはいかがでしょうか。

When using _asterisk_ routes, make sure to correctly order your routes so that _asterisk_ ones are at the end.
The route `{ path; '*' }` is usually used to 404 client side. If you are using _History mode_, make sure to [correctly configure your server](./history-mode.md) as well.
_アスタリスク_ ルートを使用するときは、_アスタリスク_ ルートが最後になるようにルートを正しく順序付けでください。
`{ path: '*' }` ルートは、通常 404 クライアントサイドで使われます。_History モード_ を使用する場合は、[正しいサーバの設定](./history-mode.md)も同様に確認してください。
Copy link
Member

Choose a reason for hiding this comment

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

ここの make sure は確認してくださいというよりも、その設定をしっかりしてくださいという意味合いだと思います。


```js
// Given a route { path: '/user-*' }
// 与えられた { path: '/user-*' } ルート
Copy link
Member

Choose a reason for hiding this comment

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

{ path: '/user-*' } というルートが与えられたとき

みたいな意味かと。

@@ -67,29 +67,29 @@ const User = {

## Catch all / 404 Not found Route
Copy link
Member

Choose a reason for hiding this comment

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

ここも訳したほうがいいですかね?

Copy link
Member Author

Choose a reason for hiding this comment

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

ah、そうですね。

@@ -4,7 +4,7 @@

**パラメータまたはクエリの変更は enter/leave ナビゲーションガードをトリガーしない** ということを覚えておいてください。それらの変更に対応するために [`$route` オブジェクトを監視する](../essentials/dynamic-matching.md#reacting-to-params-changes)、またはコンポーネント内ガード `beforeRouteUpdate` を使用するかの、どちらかができます。

## グローバルガード
## グローバルビフォーガード
Copy link
Member

Choose a reason for hiding this comment

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

これ原文は Global Guards なのでビフォーはない方がいいような……?

Copy link
Member Author

Choose a reason for hiding this comment

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

ここ、この commit で before が入ったんですよね。
c3e5872

なのでそれに従って訳している感じです。

@kazupon
Copy link
Member Author

kazupon commented Feb 8, 2019

ping @ktsn 🙏

@ktsn
Copy link
Member

ktsn commented Feb 8, 2019

@kazupon この PR が参照してるのは kazupon さんがフォークしたリポジトリのブランチだと思うのですが、そっちが更新されてないみたいです。

@kazupon
Copy link
Member Author

kazupon commented Feb 8, 2019

@ktsn
あ、すいません。今更新しました。 🙇

Copy link
Member

@ktsn ktsn left a comment

Choose a reason for hiding this comment

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

👍

@kazupon
Copy link
Member Author

kazupon commented Feb 8, 2019

@ktsn
Thanks!

@kazupon kazupon merged commit 594e8f0 into vuejs:dev Feb 8, 2019
@kazupon kazupon deleted the docs/ja branch February 8, 2019 07:05
topstack1226 pushed a commit to topstack1226/hospital-frontend that referenced this pull request Jan 3, 2023
* docs(ja): update README.md

* docs(ja): update navigation.md

ref: vuejs/vue-router@6ba8682

* docs(ja): update dynamic-matching.md

ref: vuejs/vue-router@f4d1866

* docs(ja): translate previous commit

* docs(ja): update navitation-guard.md

* Revert "docs(ja): update README.md"

This reverts commit 9b7bf986863915ffdee5ec67a86616ddad9ce046.

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)

* docs(ja): improve translate of

* docs(ja): improve translation of

* docs(ja): update

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)
SeniorDev86 added a commit to SeniorDev86/vue-router that referenced this pull request Dec 19, 2023
SeniorDev86 added a commit to SeniorDev86/vue-router that referenced this pull request Dec 19, 2023
SeniorDev86 added a commit to SeniorDev86/vue-router that referenced this pull request Dec 19, 2023
sphinxDevVic added a commit to sphinxDevVic/vue-router that referenced this pull request Feb 11, 2024
* docs(ja): update README.md

* docs(ja): update navigation.md

ref: vuejs/vue-router@6ba8682

* docs(ja): update dynamic-matching.md

ref: vuejs/vue-router@f4d1866

* docs(ja): translate previous commit

* docs(ja): update navitation-guard.md

* Revert "docs(ja): update README.md"

This reverts commit 9b7bf986863915ffdee5ec67a86616ddad9ce046.

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)

* docs(ja): improve translate of

* docs(ja): improve translation of

* docs(ja): update

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)
Luisa-m17 added a commit to Luisa-m17/VueRouter that referenced this pull request Feb 20, 2024
* docs(ja): update README.md

* docs(ja): update navigation.md

ref: vuejs/vue-router@6ba8682

* docs(ja): update dynamic-matching.md

ref: vuejs/vue-router@f4d1866

* docs(ja): translate previous commit

* docs(ja): update navitation-guard.md

* Revert "docs(ja): update README.md"

This reverts commit 9b7bf986863915ffdee5ec67a86616ddad9ce046.

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)

* docs(ja): improve translate of

* docs(ja): improve translation of

* docs(ja): update

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)
UltraDEV007 added a commit to UltraDEV007/vue-router that referenced this pull request Apr 26, 2024
* docs(ja): update README.md

* docs(ja): update navigation.md

ref: vuejs/vue-router@6ba8682

* docs(ja): update dynamic-matching.md

ref: vuejs/vue-router@f4d1866

* docs(ja): translate previous commit

* docs(ja): update navitation-guard.md

* Revert "docs(ja): update README.md"

This reverts commit 9b7bf986863915ffdee5ec67a86616ddad9ce046.

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)

* docs(ja): fix typo

ref: vuejs/vue-router#2602 (comment)

* docs(ja): improve translate of

* docs(ja): improve translation of

* docs(ja): update

ref: vuejs/vue-router#2602 (comment)
ref: vuejs/vue-router#2602 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants