Skip to content

Commit

Permalink
refactor(material/icon): removes important and inline styles
Browse files Browse the repository at this point in the history
Updates previous fix to remove the inline targets and the usage
of forced usage of styles as it is on the app to remove any
overriding styles.

Fixes b/250063405

BREAKING CHANGE: removes use of forced styles

Updates the previous fix which uses "important" and removes it
as any overrides should be fixed by the app mat-icon is used in
instead.
  • Loading branch information
essjay05 committed Oct 7, 2024
1 parent 1901dea commit 2692e51
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions src/material/icon/icon.scss
Original file line number Diff line number Diff line change
Expand Up @@ -74,11 +74,10 @@ mat-icon {
.mat-mdc-form-field .mat-icon,
.mat-grid-tile .mat-icon,
.mat-mdc-list-item .mat-icon,
.mat-mdc-list-option .mat-icon,
.mat-icon:has(div[style*='letter-spacing:']):has(div[style*='word-spacing:']) {
.mat-mdc-list-option .mat-icon {
overflow: visible;
// Apply styles to avoid text spacing
letter-spacing: 0 !important;
word-spacing: 0 !important;
line-height: 1 !important;
letter-spacing: 0;
word-spacing: 0;
line-height: 1;
}

0 comments on commit 2692e51

Please sign in to comment.