patternfly / patternfly/patternfly-react
Page: tabIndex default value (mainTabIndex=-1) which prevents keyboard scroll
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 862
- Forks
- 392
- Merge moyen
- 4 j 8 h
- PR mergées (30 j)
- 9
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par la pull request #9140 de PatternFly React référencée et le commentaire d’issue qui lui est lié afin de localiser la valeur par défaut de mainTabIndex. Supprimez le comportement tabindex par défaut, puis vérifiez que le défilement au clavier fonctionne pour les conteneurs non focalisables sans nécessiter de workaround avec mainTabIndex.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react, typescript
- Domaine
- accessibility, frontend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100