Skip to content

Conversation

@shioyang
Copy link
Collaborator

tutorial/top-ph5.md を日本語に翻訳しましたので、レビューお願いします。

issueへのリンク
#61


<figure>

<img src='generated/images/guide/toh/nav-diagram.png' alt="View navigations">
Copy link
Collaborator

Choose a reason for hiding this comment

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

altの中を翻訳していただけますか?


An Angular best practice is to load and configure the router in a separate, top-level module
that is dedicated to routing and imported by the root `AppModule`.
Angularのベストプラクティスは、ルーティングのみを行いルートの`AppModule`からインポートされる独立したトップレベルのモジュールにて、ルーターをロードし管理することです。
Copy link
Collaborator

Choose a reason for hiding this comment

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

Angularのベストプラクティスは、ルートの`AppModule`からインポートされるルーティング専用のトップレベルモジュールで、ルーターをロードして管理することです。


`--flat` puts the file in `src/app` instead of its own folder.<br>
`--module=app` tells the CLI to register it in the `imports` array of the `AppModule`.
`--flat`は、コマンドが実行されたフォルダの代わりに、`src/app`に生成したファイルを置きます。<br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

--flatは、固有のフォルダの代わりに、src/appに生成したファイルを置きます。


You generally don't declare components in a routing module so you can delete the
`@NgModule.declarations` array and delete `CommonModule` references too.
通常、ルーティング・モジュールでコンポーネントの宣言はしないので、`@NgModule.declarations`配列と`CommonModule`への参照は削除できます。
Copy link
Collaborator

Choose a reason for hiding this comment

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

ルーティングモジュール としましょう

### _RouterModule.forRoot()_

まず始めにルーターの初期化が必要で、ルーターがブラウザのロケーションの変化を検知し始めます。
You first must initialize the router and start it listening for browser location changes.
Copy link
Collaborator

Choose a reason for hiding this comment

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

原文が残っています

Copy link
Collaborator

Choose a reason for hiding this comment

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

まず始めにルーターの初期化をおこない、ルーターにブラウザのロケーション変化の検知を始めさせます。

* さまざまなコンポーネント間を行き来するためにAngularルーターを追加した。
* `AppComponent`を、リンクと`<router-outlet>`を持つナビゲーション・シェルに変更した。
* `AppRoutingModule`でルータを設定した。
* シンプルなルート、リダイレクトするルート、およびパラメータ化されたルートを定義した。
Copy link
Collaborator

Choose a reason for hiding this comment

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

パラメータ付きルートを定義しました

* `AppComponent`を、リンクと`<router-outlet>`を持つナビゲーション・シェルに変更した。
* `AppRoutingModule`でルータを設定した。
* シンプルなルート、リダイレクトするルート、およびパラメータ化されたルートを定義した。
* アンカー要素で`routerLink`ディレクティブを使用した。
Copy link
Collaborator

Choose a reason for hiding this comment

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

使用しました

* `AppRoutingModule`でルータを設定した。
* シンプルなルート、リダイレクトするルート、およびパラメータ化されたルートを定義した。
* アンカー要素で`routerLink`ディレクティブを使用した。
* 密接に結合されたマスター/詳細ビューをルート化した詳細ビューにリファクタリングした。
Copy link
Collaborator

Choose a reason for hiding this comment

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

master/detailビューをルート化した詳細ビューにリファクタリングしました

* シンプルなルート、リダイレクトするルート、およびパラメータ化されたルートを定義した。
* アンカー要素で`routerLink`ディレクティブを使用した。
* 密接に結合されたマスター/詳細ビューをルート化した詳細ビューにリファクタリングした。
* ユーザーが選択したヒーローの詳細ビューに移動するために、ルーター・リンクのパラメータを使用した。
Copy link
Collaborator

Choose a reason for hiding this comment

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

ルーターリンク 使用しました

* アンカー要素で`routerLink`ディレクティブを使用した。
* 密接に結合されたマスター/詳細ビューをルート化した詳細ビューにリファクタリングした。
* ユーザーが選択したヒーローの詳細ビューに移動するために、ルーター・リンクのパラメータを使用した。
* 複数のコンポーネント間で`HeroService`を共有した。
Copy link
Collaborator

Choose a reason for hiding this comment

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

共有しました

@lacolaco
Copy link
Collaborator

翻訳ありがとうございます!いくつかの部分で修正をお願いします 🙇

@shioyang
Copy link
Collaborator Author

レビューとコメントありがとうございます。
確認して修正します。

@shioyang
Copy link
Collaborator Author

@lacolaco コメントを確認して、修正しました。再度レビューをお願いします。

@lacolaco
Copy link
Collaborator

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

@lacolaco lacolaco merged commit 842e66c into angular:master Nov 24, 2017
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