microsoft/microsoft-ui-xaml

[ColorPicker] R/G/B or H/S/V fields should not accept alphabetical characters - move to NumberBox?

Open

#3 420 ouverte le 13 oct. 2020

Voir sur GitHub
 (9 commentaires) (1 réaction) (0 assignés)C# (659 forks)batch import
accessibilityarea-ColorPickerhelp wantedteam-Controls

Métriques du dépôt

Stars
 (5 815 stars)
Métriques de merge PR
 (Merge moyen 5j 15h) (9 PRs mergées en 30 j)

Description

Describe the bug The input fields in the ColorPicker control should only accept numerical values, not alphabetical values. When entering an alphabetical value and tabbing to the next inputfield will remove the incorrect character.

This is fine. However, Windows Narrator will still announce the alphabetical character and will not let the (visually impaired) user know that the value was incorrect.

We are running into this issue with PowerToys Settings: https://github.com/microsoft/PowerToys/issues/7085

Steps to reproduce the bug

  1. Turn on Narrator
  2. Navigate (with tab) to the Red field.
  3. Enter character (e.g. 'a')
  4. Narrator says A
  5. Tab to the next field.

Expected behavior Since these are numberfields, why not use the NumberBox control? This would be more approriate (we can set the min and max as well!) and improves accesbility.

Screenshots image

Guide contributeur

Direction de recherche
Identifiez les champs de saisie du contrôle ColorPicker qui acceptent du texte et remplacez les par des contrôles NumberBox, en veillant à définir des valeurs minimales/maximales correctes et à garantir l'accessibilité. Examinez la logique de validation actuelle et mettez la à jour pour utiliser la validation intégrée de NumberBox.
Stack technique
csharp
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#XAML
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70