DiamondLightSource / DiamondLightSource/sci-react-ui

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

Offen
#278 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
needs-triage
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.