CommunityToolkit/WindowsCommunityToolkit

RangeSelector StepFrequency doesn't behave properly at larger values

Open

#3.476 geöffnet am 14. Sept. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)C# (1.403 Forks)batch import
bug :bug:good first issue

Repository-Metriken

Stars
 (5.708 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie die Einrastlogik in RangeSelector. Die StepFrequency Eigenschaft scheint einen Fehler zu haben, wenn der Wert größer als 30 ist. Überprüfen Sie die Berechnung der Einrastpunkte und stellen Sie sicher, dass sie für jede Schrittfrequenz innerhalb des Bereichs gleichmäßig verteilt sind.
Tech Stack
csharp
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#UWP
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70