palantir/blueprint

NumericInput - prevent exceeding min or max when typing or pasting values

Open

#2071 aperta il 2 feb 2018

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
Package: coreType: enhancementhelp wanted

Metriche repository

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

Descrizione

Current NumericInput implementation caps properly at bounds when using the mouse, but when one types or pastes a value, the input should prevent typing a value exceeding the limits, it currently does not and users can type garbage.

One way to prevent this is, until a solution appears, is to check again the bounds in the onValueChange event, but is redundant.

Guida contributor

Direzione di ricerca
Esamina il gestore di eventi onValueChange del componente NumericInput nel codice sorgente di Blueprint (probabilmente in packages/core/src/components/forms/numericInput*.ts). L'implementazione esistente applica i limiti min/max per le interazioni del mouse; replica quella logica per l'input digitato/incollato. Assicurati che la convalida avvenga prima dell'aggiornamento dello stato, impedendo valori al di fuori dell'intervallo. Verifica se esiste una convalida onBlur esistente che potrebbe duplicare questo sforzo.
Tech stack
typescriptreact
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
GitReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70