patternfly / patternfly/patternfly-react
Drawer - Refine Drawer Usage for Better Integration with External Components
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 problem
This arose from fixing a layout bug with the drawer component in PR #4142.
Using DrawerPanelContent within an externally provided DrawerPanelContent (in this case from PF Topology) breaks the layout of the sidebar panel.
- Addressed by removing the nested
DrawerPanelContentand directly applyingDrawerHeadandDrawerPanelBodywithin theDrawerPanelContentthat is externally provided from TopologyView.
In our use case, the utility class pf-v6-u-p-md was applied to get the same padding, but this workaround highlights a gap in the current implementation.
How do you reproduce the problem?
See https://github.com/opendatahub-io/odh-dashboard/pull/4142.
Expected behavior
PatternFly needs an improved way to structure and compose standard drawer elements like DrawerHead and DrawerPanelBody within an external DrawerPanelContent wrapper from the TopologyView.
Is this issue blocking you? No.
What is your product and what release date are you targeting?
RHOAI
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
Empieza con PR #4142 y el uso que hace TopologyView de un DrawerPanelContent proporcionado externamente. Lee cómo se componen DrawerHead y DrawerPanelBody dentro de los componentes del drawer y, después, reproduce el escenario de diseño de la sidebar. Se considera terminado cuando los elementos estándar del drawer se integran con el wrapper externo sin el workaround de nested-content ni una clase de padding manual.
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
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100