Skip to content

Commit

Permalink
fix(material/list): not working correctly when list item is used as a…
Browse files Browse the repository at this point in the history
… button

Based on our selector for `mat-list-item`, we allow it to be a `button`, however we don't do the necessary resets for it to be usable.
  • Loading branch information
crisbeto committed Mar 29, 2022
1 parent 23f3929 commit b62ed3e
Showing 1 changed file with 21 additions and 19 deletions.
40 changes: 21 additions & 19 deletions src/material/list/list.scss
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,27 @@ $item-inset-divider-offset: 72px;
}
}

// We need some extra resets when the list items are buttons.
button.mat-list-item, button.mat-list-option {
padding: 0;
width: 100%;
background: none;
color: inherit;
border: none;
font: inherit;
outline: inherit;
-webkit-tap-highlight-color: transparent;
text-align: left;

[dir='rtl'] & {
text-align: right;
}

&::-moz-focus-inner {
border: 0;
}
}

// This mixin adjusts the heights and padding based on whether the list is in dense mode.
@mixin subheader-spacing($top-padding, $base-height) {
height: $base-height;
Expand Down Expand Up @@ -270,25 +291,6 @@ $item-inset-divider-offset: 72px;
}

mat-action-list {
// Remove the native button look and make it look like a list item
button {
background: none;
color: inherit;
border: none;
font: inherit;
outline: inherit;
-webkit-tap-highlight-color: transparent;
text-align: left;

[dir='rtl'] & {
text-align: right;
}

&::-moz-focus-inner {
border: 0;
}
}

.mat-list-item {
cursor: pointer;
outline: inherit;
Expand Down

0 comments on commit b62ed3e

Please sign in to comment.