patternfly / patternfly/patternfly-react

Page: tabIndex default value (mainTabIndex=-1) which prevents keyboard scroll

Abierto
#5,920 4 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

A11y Breaking change :boom:
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

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.