Tearsheet: Escape should only close the active topmost layer
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- frontend
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- TypeScript
- Stelle
- 10
- Fork
- 40
- Merge medio
- 23h 23m
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di patternfly/react-component-groups
-
SkeletonTable: add option to hide header skeleton rowsForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
patternfly/react-component-groups#968 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
-
Tearsheet layout updates: Masthead coverage and inset token alignmentForse già presa @GAUNSD l’ha presa 6 giorni fa. ApertaPF Team
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
patternfly/react-component-groups#985 · 1 commento · 1 assegnatario ·
-
Pull in Modal animation support for TearsheetsForse già presa @thatblindgeye l’ha presa 20 giorni fa. ApertaPF Team
patternfly/react-component-groups#979 · 1 assegnatario ·
-
MissingPage: support multiple action buttonsForse già presa @platex-rehor-bot l’ha presa 41 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
Tutte le issue di patternfly/react-component-groups
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
Signals (Failure Detector): a tool call and its own execution are reported as a repeated callAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
platformatic/mcp#208 ·
I maintainer di solito rispondono entro 1 giorno
-
🐛 bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
margelo/react-native-vision-camera#4211 ·
I maintainer di solito rispondono entro 4 giorni