DiamondLightSource / DiamondLightSource/sci-react-ui
Change Proposal: Add an accessible error state to NumberInput for other components to react to
- Lingua principale
- TypeScript
- Stelle
- 8
- Fork
- 3
- Merge medio
- 3g 15h
- PR unite (30g)
- 5
Descrizione
## What is being proposed?
As picked up by [Atlas #143](https://github.com/DiamondLightSource/atlas/pull/143#issuecomment-5207977415), the `NumberInput` currently shows user-entered invalid values, but does not commit them. As such, any form populated and used for further action may use outdated values.
## Why is this needed?
Current behaviour of components which act on inputs from `NumberInput` is unpredictable for the user, as invalid values are shown as true to a user, but not committed. This could be tracked by an accessible error state which other components can use to disable on an invalid entry.
## What will change?
Depending on implementation methods, this may change component interface, behaviour of committing an invalid value, etc.
## Interface changes (if any)
Potentially, but undetermined.
## Breaking change?
- [ ] Yes
- [ ] No
## Next steps
A maintainer will review this issue.
If accepted, it will be marked as `accepted` and a PR may then be opened.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia esaminando il componente NumberInput e la sua gestione attuale dei valori non validi inseriti dall’utente. Definisci come dovrebbe essere esposto uno stato di errore accessibile e come gli altri componenti possano reagire a esso; l’issue non identifica test né un’interfaccia definita, pertanto i criteri di completamento richiedono l’accordo dei maintainer.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- accessibility, frontend
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Tranquilla
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 35/100