patternfly / patternfly/patternfly-react
cleanup - DataListAction - id, aria-label and aria-labelledby are not currently in use but are required
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
id can probably remain (and if so should be made non-required) but aria-label and aria-labelledby should be removed because they don't usually exist on a div without a role. These props were a holdover from when we were using a Button internally opposed to children.
Jira Issue: PF-107
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
Suchen Sie die DataListAction-Komponente und prüfen Sie ihre Prop-Definitionen und Verwendungen. Bestätigen Sie, ob id noch benötigt wird und optional sein kann, und entfernen Sie dann die nicht verwendeten aria-label- und aria-labelledby-Props. Erledigt ist dies, wenn die öffentlichen Props und die zugehörigen Referenzen konsistent die aktuelle children-basierte Implementierung der Komponente widerspiegeln.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- accessibility, frontend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100