Change material icons for material design icons #346

mikesmichael · 2019-06-06T18:56:24Z

Please check if the PR fulfills these requirements

The commit message follows our guidelines:https://github.com/infra-geo-ouverte/igo2/blob/master/.github/CONTRIBUTING.md#git-commit-guidelines
Tests for the changes have been added (for bug fixes / features)
Docs have been added / updated (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior?

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No
[x] Maybe

If this PR contains a breaking change, please describe the impact and migration path for existing applications:

Some icons have new name and other have been changed with a refactoring of their display condition. This change can affect the display of icons.

Changing the name for the non-alias naming convention or fixing the display condition can resolve the issue.

Other information:

Change name for broken icon Remove Material Icon

mikesmichael requested a review from mbarbeau June 6, 2019 18:56

mikesmichael assigned mbarbeau Jun 6, 2019

mbarbeau approved these changes Jun 6, 2019

View reviewed changes

mikesmichael added 4 commits June 7, 2019 10:55

Add angular-material icon

2374177

Change all mat-icon to use svIcon for material design icon

8cd00e1

Change name for broken icon Remove Material Icon

Fix lint

d5ce650

Fix icon name

4bf6759

mbarbeau force-pushed the materialdesignicons branch from fac612c to 4bf6759 Compare June 7, 2019 14:55

mbarbeau merged commit dc7bb9d into 1.0.0-alpha.1 Jun 7, 2019

mbarbeau deleted the materialdesignicons branch June 7, 2019 14:56

Provide feedback