DiamondLightSource / DiamondLightSource/sci-react-ui
Make "read-only" input more obvious
- Vorherrschende Sprache
- TypeScript
- Sterne
- 8
- Forks
- 3
- Ø Merge
- 3 T. 15 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
[Atlas Issue #146:](https://github.com/DiamondLightSource/atlas/issues/146) For the i15-1 puck selection a component was made read-only ([see discussion](https://github.com/DiamondLightSource/atlas/pull/133#discussion_r3721451891)) but it's not obvious to a user that this is read-only at a glance. We should make read only components more visually distinctive
### What is the difference between Disabled and Read-only?
In a nut shell:
• **Disabled**: field doesn’t apply right now (irrelevant). It is removed from tab order (skipped by assistive technologies), not submitted as data. Example: when a field is inactive, because you need to select something else before this one is relevant, or, when items are being loaded into a list (disabled until active).
• **Read-only**: value still matters, just locked. Stays focusable, announced, and in tab order. The user needs to read it, not just see it greyed out. With a normal text field,`read-only` means you can select it or copy text from it (with Select I'm not so sure). Example: the current state of an item is locked, but current selection is viable information, or, see the status assigned by another user, but you cannot change it yourself.
Beitragsleitfaden
Rechercherichtung
Beginne mit der i15-1 puck-selection component und prüfe Atlas Issue #146 sowie die verlinkte PR-Diskussion zum bestehenden read-only-Verhalten. Mache read-only-Komponenten visuell unterscheidbar, ohne sie als disabled zu behandeln, und verifiziere anschließend, dass der Wert weiterhin fokussierbar und angesagt wird und in der Tab-Reihenfolge bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- accessibility, design, frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 56/100