Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upfeat(license): add license component to display full license #598
Conversation
|
Nice to see a dedicated page for the license, but there's a couple of small nitpicks that should be fixed before it can be merged. |
| @@ -0,0 +1,21 @@ | |||
| <div class="license-content"> | |||
| <p class="license-title">The MIT License</p> | |||
| <p>Copyright (c) 2010-2019 Google LLC. <a href="license">http://material.angular.io/license</a></p> | |||
EdricChan03
May 28, 2019
Contributor
Would be a good idea to use routerLink here instead of a href tag
Would be a good idea to use routerLink here instead of a href tag
| @@ -4,6 +4,7 @@ import {MatButtonModule} from '@angular/material/button'; | |||
| import {FooterModule} from '../../shared/footer/footer'; | |||
| import {RouterModule} from '@angular/router'; | |||
| import {ComponentPageTitle} from '../page-title/page-title'; | |||
| import { LicenseModule } from '../license/license'; | |||
EdricChan03
May 28, 2019
Contributor
Please remove the spaces surrounding the import.
Suggested change
import { LicenseModule } from '../license/license';
import {LicenseModule} from '../license/license';
Please remove the spaces surrounding the import.
| import { LicenseModule } from '../license/license'; | |
| import {LicenseModule} from '../license/license'; |
| @@ -0,0 +1,16 @@ | |||
| import { Component, NgModule } from "@angular/core"; | |||
EdricChan03
May 28, 2019
•
Contributor
This file should be indented with 2 spaces instead of 4 and use single quotes instead of double quotes as well as removing the spaces from your import statements.
This file should be indented with 2 spaces instead of 4 and use single quotes instead of double quotes as well as removing the spaces from your import statements.
| @@ -0,0 +1,16 @@ | |||
| @import '../../../styles/constants'; | |||
EdricChan03
May 28, 2019
Contributor
This file should be indented with 2 spaces instead of 4.
This file should be indented with 2 spaces instead of 4.
| @@ -1,5 +1,7 @@ | |||
| import {Component, NgModule} from '@angular/core'; | |||
| import {materialVersion} from '../version/version'; | |||
| import { RouterModule } from '@angular/router'; | |||
EdricChan03
May 28, 2019
Contributor
Please remove the spaces surrounding the import.
Suggested change
import { RouterModule } from '@angular/router';
import {RouterModule} from '@angular/router';
Please remove the spaces surrounding the import.
| import { RouterModule } from '@angular/router'; | |
| import {RouterModule} from '@angular/router'; |
| @@ -14,6 +16,7 @@ export class Footer { | |||
|
|
|||
|
|
|||
| @NgModule({ | |||
| imports: [RouterModule.forRoot(MATERIAL_DOCS_ROUTES)], | |||
EdricChan03
May 28, 2019
Contributor
Please use RouterModule#forChild instead of RouterModule#forRoot on submodules/lazy-loaded modules (as indicated in the API docs for RouterModule).
Suggested change
imports: [RouterModule.forRoot(MATERIAL_DOCS_ROUTES)],
imports: [RouterModule.forChild(MATERIAL_DOCS_ROUTES)],
Please use RouterModule#forChild instead of RouterModule#forRoot on submodules/lazy-loaded modules (as indicated in the API docs for RouterModule).
| imports: [RouterModule.forRoot(MATERIAL_DOCS_ROUTES)], | |
| imports: [RouterModule.forChild(MATERIAL_DOCS_ROUTES)], |
| @@ -1,5 +1,7 @@ | |||
| import {Component, NgModule} from '@angular/core'; | |||
| import {materialVersion} from '../version/version'; | |||
| import { RouterModule } from '@angular/router'; | |||
| import { MATERIAL_DOCS_ROUTES } from '../../routes'; | |||
EdricChan03
May 28, 2019
Contributor
Please remove the spaces surrounding the import.
Suggested change
import { MATERIAL_DOCS_ROUTES } from '../../routes';
import {MATERIAL_DOCS_ROUTES} from '../../routes';
Please remove the spaces surrounding the import.
| import { MATERIAL_DOCS_ROUTES } from '../../routes'; | |
| import {MATERIAL_DOCS_ROUTES} from '../../routes'; |
Adding the full license aggreement as per angular.io example in reference to angular/components#16039