CommunityToolkit/WindowsCommunityToolkit

RangeSelector StepFrequency doesn't behave properly at larger values

Open

#3.476 aberto em 14 de set. de 2020

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)C# (1.403 forks)batch import
bug :bug:good first issue

Métricas do repositório

Stars
 (5.708 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Describe the bug

In the sample app, with the example range between 0-100, values over 30 (40, 50) etc... don't act properly.

For instance, 30 snaps at 30 and 60 but not 90. 40 snaps at 40, but not 80 60 doesn't snap...

Steps to Reproduce

Steps to reproduce the behavior:

  1. Go to RangeSelector sample in sample app.
  2. Go to XAML tab
  3. Set StepFrequency to a custom value
  4. See behavior above

Expected behavior

30 should still snap at 30, 60, 90 40 should snap at 40 & 80 60 should snap at 60

Environment

Sample App

Windows 10 Build Number:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] May 2019 Update (18362)
- [x] May 2020 Update (19041)
- [ ] Insider Build (build number: )

Device form factor:
- [x] Desktop
- [ ] Xbox
- [ ] Surface Hub
- [ ] IoT

Additional context

Found when investigating #3473, FYI @hansmbakker

Guia do colaborador

Direção de pesquisa
Inspecione a lógica de snap no RangeSelector. A propriedade StepFrequency parece ter um bug quando o valor é maior que 30. Verifique o cálculo dos pontos de snap e garanta que eles sejam espaçados uniformemente para qualquer frequência de passo dentro do intervalo.
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#UWP
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70