patternfly / patternfly/patternfly-react

Drawer - Refine Drawer Usage for Better Integration with External Components

Abierto
#11,804 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Stale
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 DrawerPanelContent and directly applying DrawerHead and DrawerPanelBody within the DrawerPanelContent that 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

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

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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.