Chip: on the web every Chip sets aria-selected on role button

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#5,180 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
84/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
react-native, typescript
Bereich
accessibility, frontend

Rechercherichtung

Beginne in Chip.tsx, wo laut Issue aria-selected übergeben wird und selected standardmäßig auf false gesetzt ist; vergleiche das Verhalten im Web mit dem nativen Verhalten, bevor du etwas änderst. Prüfe das Chip-Beispiel unter /chip und führe axe-core für die Web-App aus, um zu bestätigen, dass der Verstoß behoben ist und ein ausgewählter Toggle seinen Zustand offenlegt. Fertig, wenn einfache Aktions-Chips keinen Toggle-Zustand angeben und ausgewählte Web-Chips den unterstützten Button-Zustand verwenden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Accessibility bug web
Current behaviour

On the web, every Chip renders <button role="button" aria-selected="true|false"> (Chip.tsx passes aria-selected={selected}, and selected defaults to false). aria-selected is not allowed on role button, so screen readers ignore the selected state, and axe-core reports aria-allowed-attr (critical): "ARIA attribute is not allowed: aria-selected". The example app shows it on the Chip, Card, Floating Action Button, List.Section and Tooltip screens.

Expected behaviour

On the web a Chip used as a toggle exposes its state with aria-pressed, the state role button supports, and only when the app passes selected, so a plain action chip isn't announced as a toggle. Native keeps announcing the selected state as it does today. axe reports no aria-allowed-attr violation.

How to reproduce?
  1. Run the example app on the web from main (yarn install, then npx expo start --web in example/).
  2. Open the Chip example (/chip).
  3. Run axe-core on the page (for example with the axe DevTools extension).
Preview
What have you tried so far?
Your Environment
software version
react-native-paper main (8cb344b)
browser Chromium 153
Vorherrschende Sprache
TypeScript
Sterne
14.5k
Forks
2.3k
Ø Merge
2 T. 12 Std.
Gemergte PRs (30 T.)
22

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus callstack/react-native-paper

Alle Issues in callstack/react-native-paper

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

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