patternfly / patternfly/patternfly-react
Page: tabIndex default value (mainTabIndex=-1) which prevents keyboard scroll
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 862
- Forks
- 392
- Merge medio
- 4 d 8 h
- PR fusionados (30 d)
- 9
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el pull request #9140 de PatternFly React mencionado y el comentario de issue vinculado para localizar el valor predeterminado de mainTabIndex. Elimina el comportamiento predeterminado de tabindex y verifica después que el desplazamiento mediante el teclado funcione para contenedores no enfocables sin requerir un workaround con mainTabIndex.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- react, typescript
- Área
- accessibility, frontend
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100