Button - update console error for accessible name and move aria-expanded location
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 55/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- react, typescript
- Bereich
- accessibility, frontend
Rechercherichtung
Beginne mit der Prüfung der referenzierten Änderungen in pull request #11861, insbesondere des accessible-name-Konsolenfehlers um Zeile 164 und der aria-expanded-Behandlung um Zeile 239. Finde die im Issue erwähnten zugehörigen Button-Tests, aktualisiere sie für das angeforderte Verhalten und bestätige, dass die Fehler- und Attributbehandlung den angegebenen Anforderungen entspricht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the enhancement or change
Followup to some comments left in https://github.com/patternfly/patternfly-react/pull/11861/
- The console error with the
isSettings || isHamburger || isFavoriteconditional for the accessible name (line 164 in that PR) should be updated to remove that conditional. We should be throwing an error for any button that doesn't have children, aria-label, or aria-labelledby - The
aria-expandedattribute (line 239) should be moved below the{...props}so that it won't be overridden by a manualaria-expandedbeing passed in - Several tests related to the above points will need to be updated in some fashion as well
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?
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus patternfly/patternfly-react
-
PF Team
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
patternfly/patternfly-react#12651 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
patternfly/patternfly-react#12637 ·
-
PF Team
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
patternfly/patternfly-react#12617 · 1 Kommentar ·
-
PF Team
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
patternfly/patternfly-react#12610 · 1 Kommentar · 1 Reaktion ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
patternfly/patternfly-react#12600 ·
Alle Issues in patternfly/patternfly-react
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
bcgov/bc-wallet-mobile#4761 · 1 Kommentar ·
-
external-issue to-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
-
area-deployment area-integrations triage:bot-seen
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 86/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
refactor
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100