Tearsheet: Escape should only close the active topmost layer
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Start from the Tearsheet and modal components that handle Escape-key events, then reproduce the nested-modal and stacked-Tearsheet cases described in the issue. Trace which layer receives the event and inspect existing tests around modal dismissal; done means Escape closes only the active topmost layer while underlying layers remain open.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
When a Tearsheet opens another modal, such as a file-selection modal, pressing
Escape closes both the modal and the underlying Tearsheet.
This suggests that the Escape-key event is being handled by multiple layers rather
than only by the active topmost layer. With stacked Tearsheets, the same behavior
could dismiss multiple workflow levels with a single keypress and unexpectedly
discard in-progress work.
Steps to reproduce
-
Open a Tearsheet.
-
Open a modal or file-selection flow from within the Tearsheet.
-
Press
Escape. -
Observe that both the nested modal and the underlying Tearsheet close.
-
Repeat with multiple stacked Tearsheets and verify whether one
Escapedismisses more than the active layer.
Here I hit escape to cancel out of the modal but all views are closed:
Expected behavior
Pressing Escape should dismiss only the active topmost modal or Tearsheet.
Underlying layers should remain open, and the event should not continue to
trigger their close handlers.
Context
This was observed in
odh-dashboard#9589.
The follow-up discussion is
here.
Related stacked-modal accessibility work:
Jira Issue: PF-4706
- Lenguaje dominante
- TypeScript
- Estrellas
- 10
- Forks
- 40
- Merge medio
- 23 h 23 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de patternfly/react-component-groups
-
SkeletonTable: add option to hide header skeleton rowsPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
patternfly/react-component-groups#968 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
Tearsheet layout updates: Masthead coverage and inset token alignmentPosiblemente ocupada @GAUNSD la tomó hace 6 días. AbiertoPF Team
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
patternfly/react-component-groups#985 · 1 comentario · 1 asignado ·
-
Pull in Modal animation support for TearsheetsPosiblemente ocupada @thatblindgeye la tomó hace 20 días. AbiertoPF Team
patternfly/react-component-groups#979 · 1 asignado ·
-
MissingPage: support multiple action buttonsPosiblemente ocupada @platex-rehor-bot la tomó hace 41 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Todos los issues de patternfly/react-component-groups
Issues similares
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 85/100
capricorn86/happy-dom#2474 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
JSerwatka/letterboxd-tweaks#81 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
siyuan-note/siyuan#20165 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
polkadot-js/phishing#5716 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
juice-shop/juice-shop#3662 ·
Los mantenedores suelen responder en 1 día