List.Item height

[agat]

Environment

Latest stable version of paper.

Description

Based on Material Design specs list item height must be:

• With title: 48px
• With title and icon: 56px
• With title and description: 72px

https://material.io/design/components/lists.html#specs

But now List.Item component height are:

• With title: 54px
• With title and icon: 72px
• With title and description: 72px

Reproducible Demo

<List.Item
    title="Trello"
    left={props => <List.Icon icon="folder" />}
    description="Project 1, Project 2"
/>
<List.Item
    title="Trello"
    left={props => <List.Icon icon="folder" />}
/>
<List.Item
    title="Trello"
/>

[ocarreterom]

Yes, List.Item lacks some variants.

[ferrannp]

@agat wanna make a PR?

[Taym95]

If no one working on this issue, I would like to work on it.

[agat]

There is a desire to try to do it, but there is no time in the coming days.

[Trancever]

@Taym95 Fell free to submit PR :)

[smkhalsa]

@Taym95 Have you had a chance to take a stab at this?

[mitulsavani]

I would like to give a shot on this issue too!
Can somebody quickly tell me how I can test the component after making changes?

[Trancever]

@mitulsavani There is an Example app in Example directory. For more informations check Contributing Guide

[Trancever]

Fix is included into 3.0.0-alpha.0.