Drawer fork lets parent contexts leak in (no LayerContentBoundary)

Abierto
#89 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
42/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
react, typescript
Área
frontend

Línea de trabajo

Start with src/forks/Drawer/Drawer.tsx and compare its context behavior with Astryx 0.7.2's core Drawer, which wraps content in LayerContentBoundary. The issue gives two possible directions: request an exported boundary or move Modal and the notification stack onto the top layer so the fork can be removed. First determine which path maintainers choose; done means parent contexts no longer leak into drawer content, with the upstream test from #88 enabled or an equivalent check.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Since Astryx 0.7.2, core's Drawer wraps its content in a private LayerContentBoundary, so contexts such as a disabled ButtonGroup or SizeContext around the trigger do not leak into the drawer. ui-common's Drawer fork (src/forks/Drawer/Drawer.tsx) cannot import that boundary, so those contexts still reach the drawer's content, as they did with lab's Drawer. The upstream test for it is skipped in #88.

Options:

  • ask Astryx to export LayerContentBoundary (or an equivalent reset), then use it in the fork;
  • move ui-common's Modal and the notification stack onto the top layer so the fork's reasons go away and it can be deleted in favour of core's Drawer.
Lenguaje dominante
TypeScript
Estrellas
2
Forks
0
Merge medio
1 d 1 h
PR fusionados (30 d)
35

Preparar el entorno

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.

Más de lablup/ui-common

Todos los issues de lablup/ui-common

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.