Tearsheet: Escape should only close the active topmost layer
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 68/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- typescript
- Bereich
- frontend
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- TypeScript
- Sterne
- 10
- Forks
- 40
- Ø Merge
- 12 Min.
- Gemergte PRs (30 T.)
- 1
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus patternfly/react-component-groups
-
SkeletonTable: add option to hide header skeleton rowsEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
patternfly/react-component-groups#968 · 1 Kommentar ·
-
ColumnManagementModal: add the ability to have sticky columnsEvtl. vergeben @logonoff hat das heute übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
-
Tearsheet layout updates: Masthead coverage and inset token alignmentEvtl. vergeben @GAUNSD hat das vor 7 Tagen übernommen. OffenPF Team
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
patternfly/react-component-groups#985 · 1 Kommentar · 1 zugewiesene Person ·
-
Pull in Modal animation support for TearsheetsEvtl. vergeben @thatblindgeye hat das vor 21 Tagen übernommen. OffenPF Team
patternfly/react-component-groups#979 · 1 zugewiesene Person ·
-
MissingPage: support multiple action buttonsEvtl. vergeben @platex-rehor-bot hat das vor 42 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
Alle Issues in patternfly/react-component-groups
Ähnliche Issues
-
DB-plane provider_chat_options.* is accepted by config set but never merged into the loaded configOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
Bump Firebase JS SDK (12.19.0 → 13.0.0)Evtl. vergeben @SelaseKay hat das heute übernommen. OffenNeeds Attention type: enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
invertase/react-native-firebase#9364 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
clouflaure de fernandoOffenenhancement
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
cloudflare/mcp#271 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 4 Tagen
-
e2e-failure ready-to-code
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
redhat-developer/rhdh-plugin-export-overlays#4261 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag