patternfly / patternfly/patternfly-react
Page - change notificationDrawer props to be more generic
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
re: https://github.com/patternfly/patternfly-react/issues/8541#issuecomment-1384624420
There are currently a props on the <Page> component (notificationDrawer and isNotificationDrawerExpanded) that attach a drawer to the page. AFAIK there is no requirement to use a notification drawer there - the props just update the page to support a drawer that appears over the main content area. Assuming we don't want to limit that use of a drawer in the page to notification drawer only, I wonder if it makes sense to rename those props to something more generic so it's clear it can be used for any sort of drawer content.
cc @mcarrano @mceledonia @tlabaj
Jira Issue: PF-141
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 en el componente Page y busca las props notificationDrawer e isNotificationDrawerExpanded y sus usos. Confirma el nombre genérico esperado para el contenido del drawer, actualiza las referencias afectadas y verifica que el comportamiento del drawer de Page no cambie.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- react, typescript
- Área
- frontend
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100