Tearsheet: Escape should only close the active topmost layer
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 68/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Ativa
- Stack de tecnologia
- typescript
- Domínio
- frontend
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 10
- Forks
- 40
- Merge médio
- 23h 23min
- PRs com merge (30d)
- 2
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de patternfly/react-component-groups
-
SkeletonTable: add option to hide header skeleton rowsTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
patternfly/react-component-groups#968 · 1 comentário ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
-
Tearsheet layout updates: Masthead coverage and inset token alignmentTalvez já em andamento @GAUNSD assumiu há 7 dias. AbertaPF Team
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
patternfly/react-component-groups#985 · 1 comentário · 1 responsável ·
-
Pull in Modal animation support for TearsheetsTalvez já em andamento @thatblindgeye assumiu há 21 dias. AbertaPF Team
patternfly/react-component-groups#979 · 1 responsável ·
-
MissingPage: support multiple action buttonsTalvez já em andamento @platex-rehor-bot assumiu há 42 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
Todas as issues de patternfly/react-component-groups
Issues semelhantes
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Small-tailqwq/dsh-deep-whale#187 ·
Mantenedores costumam responder em até 1 dia
-
Tela Marca manda conferir o campo "Razão social", que em Portugal se chama "Denominação social"Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
melgarafael/DeskcommCRM#2503 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
0x80/isolate-package#218 ·
-
bug via-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
pingdotgg/t3code#16859 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
Mantenedores costumam responder em até 1 dia