Skip to content

Commit

Permalink
fix(menu): Set menu-item icon color only when not set on mat-icon (an…
Browse files Browse the repository at this point in the history
…gular#8614)

* fix(menu): Set menu-item icon color if not set on mat-icon

Set the theme icon color only if the `color` input property was not set on the `mat-icon` component.

Fixes angular#8594

* mat-menu-item with mat-icon[color] unit tests
Add unit tests for setting the mat-icon[color] attribute on a menu item

* Make ItemIconColorsMenu more clear
Related to angular#8594

* Fix linting issues

* Fix linting

* Removed "CSS engine tests"
  • Loading branch information
glenstaes authored and tinayuangao committed Dec 1, 2017
1 parent e939ea7 commit d0cb077
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/lib/menu/_menu-theme.scss
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
}
}

.mat-menu-item .mat-icon,
.mat-menu-item .mat-icon:not([color]),
.mat-menu-item-submenu-trigger::after {
color: mat-color($foreground, 'icon');
}
Expand Down

0 comments on commit d0cb077

Please sign in to comment.