palantir/blueprint

NumericInput controlled mode warns about invalid value too often

Open

#4 327 ouverte le 10 sept. 2020

Voir sur GitHub
 (1 commentaire) (9 réactions) (0 assignés)TypeScript (2 167 forks)batch import
P2Package: coreType: enhancementhelp wanted

Métriques du dépôt

Stars
 (20 263 stars)
Métriques de merge PR
 (Merge moyen 43j 10h) (27 PRs mergées en 30 j)

Description

Environment

  • Package version(s): core 3.31.0

Steps to reproduce

  1. set up <NumericInput value={this.state.value} onChange={(_v, value) => this.setState({ value })) />
  2. type 0.

Actual behavior

in dev mode, the component complains that

<NumericInput> controlled value prop does not adhere to stepSize, min, and/or max constraints.

2020-09-10 13 38 22

Expected behavior

this should be allowed. what it actually means is that the string passed was not a full numeric entity, but it can still be parsed as a valid number, so it's ok.

Possible solution

change the validation logic

Guide contributeur

Direction de recherche
Examinez la logique de validation du composant NumericInput, probablement dans src/components/forms/numericInput.tsx ou similaire. La validation actuelle rejette des chaînes comme '0.' qui ne sont pas des entités numériques complètes mais qui sont analysées comme des nombres valides. La correction doit autoriser de telles valeurs intermédiaires lors de la saisie, tout en appliquant des contraintes sur la valeur finale. Envisagez d'utiliser une expression régulière ou une fonction d'analyse qui accepte les nombres incomplets.
Stack technique
reacttypescript
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
GitReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75