React-native-paper: List.Item height

Created on 12 Mar 2019  路  9Comments  路  Source: callstack/react-native-paper

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" />

Android Emulator - Pixel_API_25:5554 2019-03-12 13-44-04

good first task

Most helpful comment

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

All 9 comments

Yes, List.Item lacks some variants.

@agat wanna make a PR?

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

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

@Taym95 Fell free to submit PR :)

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

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?

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

Fix is included into 3.0.0-alpha.0.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

sritharanpalani picture sritharanpalani  路  4Comments

ZhengYuTay picture ZhengYuTay  路  3Comments

zachariahtimothy picture zachariahtimothy  路  3Comments

satya164 picture satya164  路  4Comments

mihaidaviddev picture mihaidaviddev  路  3Comments