Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Align ListItemIcon with ListItemText's primary #17828

Closed
2 tasks done
FranBran opened this issue Oct 10, 2019 · 3 comments
Closed
2 tasks done

Align ListItemIcon with ListItemText's primary #17828

FranBran opened this issue Oct 10, 2019 · 3 comments
Labels
component: list This is the name of the generic UI component, not the React module! design: material This is about Material Design, please involve a visual or UX designer in the process

Comments

@FranBran
Copy link

  • The issue is present in the latest release.
  • I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

According to the Material-Design-Specs the ListItemIcon should be aligned with the ListItemText's primary, but it's vertically centered instead.

grafik

Expected Behavior 🤔

grafik
@eps1lon eps1lon added component: list This is the name of the generic UI component, not the React module! design: material This is about Material Design, please involve a visual or UX designer in the process labels Oct 10, 2019
@emmanio
Copy link

emmanio commented Oct 11, 2019

I can fix this one :) will put up a PR this afternoon

@emmanio emmanio mentioned this issue Oct 11, 2019
Closed
1 task
@siriwatknp
Copy link
Member

This issue is outdated (MD2 related). We aim to have a fresh look for Material UI in the near future, so I’m closing this.

@github-actions GitHub Actions
Copy link

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

Note

@FranBran How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: list This is the name of the generic UI component, not the React module! design: material This is about Material Design, please involve a visual or UX designer in the process
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

17828 ListItemIcon alignment when secondary text is enabled
4 participants
@FranBran @eps1lon @siriwatknp @emmanio