palantir/blueprint

NumericInput controlled mode warns about invalid value too often

Open

#4327 aperta il 10 set 2020

Vedi su GitHub
 (1 commento) (9 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
P2Package: coreType: enhancementhelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga la logica di validazione del componente NumericInput, probabilmente in src/components/forms/numericInput.tsx o simile. L'attuale validazione rifiuta stringhe come '0.' che non sono entità numeriche complete ma vengono analizzate come numeri validi. La correzione dovrebbe consentire tali valori intermedi durante la digitazione, continuando a imporre vincoli sul valore finale. Considera l'uso di un'espressione regolare o di una funzione di analisi che accetti numeri incompleti.
Tech stack
reacttypescript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75