patternfly / patternfly/patternfly-react
Spike - Investigate options for updating a11y props being required or not
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 enhancement or change
Spike for a discussion to have regarding https://github.com/patternfly/patternfly-react/issues/5397
Basically we need to decide whether:
-
we want to update certain a11y props to either be required (if currently optional) or make them optional (if currently required)
-
we want to add console warnings when an optional a11y prop is not passed in, but ideally should be (e.g. if an element doesn't have a visible text label, typically we should want an
aria-labeladded somewhere) -
leave things as-is - props will remain optional or required as they already are
-
some combination of all of the above
-
some props should have an (updated?) default value
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?
Jira Issue: PF-108
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 leyendo el issue #5397 de PatternFly enlazado y revisa las props de accesibilidad obligatorias y opcionales actuales en los componentes afectados. Compara las opciones descritas aquí, incluidas las advertencias y los valores predeterminados. Se considera completado cuando las alternativas, las props afectadas y una dirección recomendada están documentadas para su discusión.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- react, typescript
- Área
- accessibility
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Tranquilo
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100