xamarin/Xamarin.Forms

[Bug] OnPlatform doesnt work with Comma separated values

Aberta

#6.048 aberto em 30 de abr. de 2019

Ver no GitHubGuia do colaborador
 (4 comentários) (0 reação) (0 responsável)C# (1.926 forks)batch import
a/Xaml </>e/1 :clock1:hacktoberfest 🍻help wantedt/bug :bug:up-for-grabs

Métricas do repositório

Stars
 (5.644 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Description

if i want to define padding or margin with multiple values onplatform like below. it doesnt work. I guess that comma is confused with comma between platform definitions.

Padding="{OnPlatform iOS=0,30,0,0, Default=0}"

Steps to Reproduce

  1. define padding on anything like Padding="{OnPlatform iOS=0,30,0,0, Default=0}"

Expected Behavior

it recognises padding or margin

Actual Behavior

no error but not working at all.

Basic Information

  • Version with issue:
  • Last known good version:
  • IDE: vs 2019
  • Platform Target Frameworks:
    • iOS: 12.2
    • Android:
    • UWP:
  • Android Support Library Version:
  • Nuget Packages:
  • Affected Devices:

xamarin forms:3.6

Guia do colaborador

Direção de pesquisa
Investigue a lógica de análise da extensão de marcação XAML para OnPlatform para lidar com valores contendo vírgulas.
Pilha de tecnologia
csharp
Domain
frontend
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#XAML
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65