patternfly / patternfly/patternfly-react
Modal - create variation using `dialog` element
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
Is this a new component or an extension of an existing one?
The Modal creates a ModalBox, which will have a variation in Core built using the <dialog> element.
Describe the feature
<dialog> provides functionality built into browsers that handles placing the modal on the top layer, handling keyboard focus, and providing a ::backdrop pseudoelement. Users should be able to opt in to this variant, which then probably does not need the separate Backdrop component, may not need to be placed in the DOM where it is now, and focus will be handled by the browser. A11y should be reviewed given that the dialog implicitly provides appropriate aria labeling.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/dialog
This issue is dependent on completion in core in https://github.com/patternfly/patternfly/issues/7296
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 revisando el comportamiento existente de React Modal y ModalBox; después, lee el core issue 7296 enlazado y la documentación de MDN sobre dialog. Comprueba cómo se utilizan actualmente el componente Backdrop y la gestión del foco. Se considera terminado cuando funciona una variante basada en dialog y opt-in con un comportamiento de accesibilidad adecuado, y se ha resuelto su interacción con Backdrop y la colocación en el DOM.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- html, react, typescript
- Área
- accessibility, frontend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100