patternfly / patternfly/patternfly-react
Spike - Investigate options for updating a11y props being required or not
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Describe the enhancement or change
Spike for a discussion to have regarding https://github.com/patternfly/patternfly-react/issues/5397
Basically we need to decide whether:
-
we want to update certain a11y props to either be required (if currently optional) or make them optional (if currently required)
-
we want to add console warnings when an optional a11y prop is not passed in, but ideally should be (e.g. if an element doesn't have a visible text label, typically we should want an
aria-labeladded somewhere) -
leave things as-is - props will remain optional or required as they already are
-
some combination of all of the above
-
some props should have an (updated?) default value
Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?
Any other information?
Jira Issue: PF-108
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 mit dem Lesen des verlinkten PatternFly-Issues #5397 und überprüfe die aktuellen erforderlichen und optionalen Accessibility-Props in den betroffenen Komponenten. Vergleiche die hier beschriebenen Optionen einschließlich Warnungen und Standardwerten. Als abgeschlossen gilt, wenn die Alternativen, die betroffenen Props und eine empfohlene Richtung zur Diskussion dokumentiert sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- accessibility
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Ruhig
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100