patternfly / patternfly/patternfly-react
Table - allow providing aria-label to the input for row selection
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Hi there 👋
Just wondering if it would make sense to allow providing a custom aria-label for the input used for row selection (radio or checkbox, depending on the TdSelectType variant). From what I can see, the label is currently hardcoded as Select row ${rowIndex} in the selectable decorator, and I couldn't find a way to customize it.
I'm not sure if I'm missing something or if overriding that label would go against intended usage, but I believe being able to provide a custom value to at least support translations might be useful in some cases.
Would this be a reasonable thing to support?
Thanks!
Jira Issue: PF-2346
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in packages/react-table/src/components/Table/utils/decorators/selectable.tsx, wo das aria-label für die Zeilenauswahl derzeit zusammengesetzt wird, und verfolge, wie TdSelectType und die Selection-Props der Tabelle diesen Decorator erreichen. Prüfe die Komponentendokumentation auf die unterstützte Selection-API. Erledigt ist die Aufgabe, wenn Nutzer ein benutzerdefiniertes Label für sowohl Checkbox- als auch Radio-Zeileneingaben bereitstellen können, während der bestehende Standard weiterhin verfügbar ist und Übersetzungen unterstützt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- accessibility, frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 68/100