xamarin/Xamarin.Forms

[Bug] Top and Bottom padding on Material button can't be set to less than 8dp

Aberta

#7.860 aberto em 7 de out. de 2019

Ver no GitHubGuia do colaborador
 (2 comentários) (2 reações) (0 responsável)C# (1.926 forks)batch import
a/visuale/3 :clock3:help wantedt/bug :bug:up-for-grabs

Métricas do repositório

Stars
 (5.644 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Description

        <StackLayout Visual="Material">
            <Button  Padding="0" Text="no padding" ></Button>
            <Button  Text="default"></Button>
        </StackLayout>

Both of the controls above render with the same padding applied to the control so you can't get a 36 dp height on a Material Button (which is allowable as part of the Material spec)

Expected Behavior

When you set padding to zero the padding on all side of the control should be zero

Actual Behavior

It's not zero

Basic Information

  • Version with issue: master (I also tested with Android X)

Guia do colaborador

Direção de pesquisa
Inspecione a implementação do renderizador do botão Material no Xamarin.Forms para encontrar onde o preenchimento mínimo (8dp) é aplicado e como torná lo configurável.
Pilha de tecnologia
csharp
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Xamarin.Forms
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65