DiamondLightSource / DiamondLightSource/sci-react-ui
Change Proposal: Add an accessible error state to NumberInput for other components to react to
- Vorherrschende Sprache
- TypeScript
- Sterne
- 8
- Forks
- 3
- Ø Merge
- 3 T. 15 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
## 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.
Beitragsleitfaden
Rechercherichtung
Beginne mit der Untersuchung der NumberInput-Komponente und ihrer aktuellen Verarbeitung ungültiger, vom Benutzer eingegebener Werte. Lege fest, wie ein barrierefreier Fehlerzustand offengelegt werden sollte und wie andere Komponenten darauf reagieren können; das Issue benennt weder Tests noch eine festgelegte Schnittstelle, daher erfordern die Abschlusskriterien die Zustimmung der Maintainer.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- accessibility, frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Ruhig
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100