DiamondLightSource / DiamondLightSource/sci-react-ui

Change Proposal: Add an accessible error state to NumberInput for other components to react to

Aperta
#278 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
needs-triage
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.