patternfly / patternfly/patternfly-react
Page: tabIndex default value (mainTabIndex=-1) which prevents keyboard scroll
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 issue. What is the expected and unexpected behavior?
mainTabIndex=-1 as a default value prevents keyboard scroll. tlabindex should be removed. Having a tabindex on a container essentially breaks focus and keyboard scroll for all non-focusable containers below it. eg topology details side panel.
Any contributions to the main content should not have to deal with adding their own tabindex attribute to their scroll containers.
Above notes from the pr: https://github.com/openshift/console/pull/9140#issuecomment-854134911
workaround is to set the mainTabIndex to null.
Please provide the steps to reproduce. Feel free to link CodeSandbox or another tool.
https://github.com/openshift/console/pull/9140#issuecomment-8541349113
Is this a bug or enhancement? If this issue is a bug, is this issue blocking you or is there a work-around?
Bug
What is your product and what release version are you targeting?
OpenShift 4.9
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 referenzierten PatternFly React Pull Request #9140 und dem verknüpften Issue-Kommentar, um den Standardwert von mainTabIndex zu finden. Entferne das standardmäßige tabindex-Verhalten und überprüfe anschließend, dass das Scrollen per Tastatur für nicht fokussierbare Container funktioniert, ohne einen mainTabIndex-Workaround zu erfordern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- accessibility, frontend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100