-
Notifications
You must be signed in to change notification settings - Fork 6.8k
fix(material-experimental/mdc-list): fix styles for normal lists #18632
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -8,3 +8,53 @@ | |||
.mat-mdc-list-base { | |||
display: block; | |||
} | |||
|
|||
// While MDC expects only span elements under mdc-list-item__text, we have historically supported |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do they have any examples where there's a header etc. outside mdc-list-item__text
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, fixed the avatar and divider issues |
Fixes up various styles for the lists under the "normal lists" section of the demo. There are still some issues around the avatars and dividers on the last list
@jelbourn updated the dev-app and examples to remove |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
) * fix(material-experimental/mdc-list): fix styles for normal lists Fixes up various styles for the lists under the "normal lists" section of the demo. There are still some issues around the avatars and dividers on the last list * fix avatars and dividers * fix lint and api * address feedback (cherry picked from commit dd239dd)
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Fixes up various styles for the lists under the "normal lists" section of the demo.
Note: not merge-safe due to some minor changes in mat-line.
Demo: https://mmalerba-demo1.firebaseapp.com/mdc-list