patternfly / patternfly/patternfly-react

Page - update logic to apply sidebar-less layout

Abierto
#11,797 0 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Breaking change :boom: PF Team
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
Followup to https://github.com/patternfly/patternfly-react/pull/11760 where we apply logic to apply a modifier class for sidebar-less layout if sidebar={null} (tldr reason being to avoid breaking consumers who may rely on sidebar being explicitly undefined, but want to keep the layout as-is).

We want to update the logic in the next breaking change so that omitting the sidebar prop (or it otherwise being falsey) will apply this modifier class instead of having to explicitly pass null to the prop.

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?

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

Revisa la lógica del diseño sin sidebar en el componente Page y el comportamiento introducido por el pull request #11760. Rastrea cómo se gestionan las props de sidebar omitidas, falsey y establecidas explícitamente en null, y después verifica que la clase modificadora se aplique en los casos previstos sin romper los consumidores existentes.

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
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.