Skip to content

Commit 4e01b84

Browse files
authored
fix(material/button): fix ripple style for all types of buttons (#26970)
1 parent 036e514 commit 4e01b84

File tree

3 files changed

+6
-12
lines changed

3 files changed

+6
-12
lines changed

src/material/button/_button-base.scss

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,12 @@
3030
border-radius: inherit;
3131
}
3232

33+
// This style used to be applied by the MatRipple
34+
// directive, which is no longer attached to this element.
35+
.mat-mdc-button-ripple {
36+
overflow: hidden;
37+
}
38+
3339
// We use ::before so that we can reuse some of MDC's theming.
3440
.mat-mdc-button-persistent-ripple::before {
3541
content: '';

src/material/button/button.scss

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,12 +92,6 @@
9292
}
9393
}
9494

95-
// This style used to be applied by the MatRipple
96-
// directive, which is no longer attached to this element.
97-
.mat-mdc-button-ripple {
98-
overflow: hidden;
99-
}
100-
10195
// Since the stroked button has has an actual border that reduces the available space for
10296
// child elements such as the ripple container or focus overlay, an inherited border radius
10397
// for the absolute-positioned child elements does not work properly. This is because the

src/material/button/icon-button.scss

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,6 @@
5555
@include button-base.mat-private-button-touch-target(true);
5656
@include private.private-animation-noop();
5757

58-
// This style used to be applied by the MatRipple
59-
// directive, which is no longer attached to this element.
60-
.mat-mdc-button-ripple {
61-
overflow: hidden;
62-
}
63-
6458
.mat-mdc-button-persistent-ripple {
6559
border-radius: 50%;
6660
}

0 commit comments

Comments
 (0)