palantir/blueprint

[v5] Popover not closing when navigating lazily loaded pages via React Router

Open

#6.518 aberto em 6 de nov. de 2023

Ver no GitHub
 (2 comments) (3 reactions) (0 assignees)TypeScript (2.167 forks)batch import
P2Package: coreType: bughelp wanted

Métricas do repositório

Stars
 (20.263 stars)
Métricas de merge de PR
 (Mesclagem média 43d 10h) (27 fundiu PRs em 30d)

Description

Environment

  • Package version(s): @blueprintjs/core 5.6.0
  • Operating System: Windows 10
  • Browser name and version: Chrome 118

Code Sandbox

https://codesandbox.io/p/sandbox/blueprint5-popover-issue-9zjj9x

Steps to reproduce

  1. Use the navbar menu to navigate to "Page 2"

Actual behavior

When using React Router to navigate to a lazily loaded page, the popover menu does not close and loses interactivity.

Expected behavior

The popover menu should close after the page transition.

Possible solution

No current solution to propose. We're currently migrating our code from Blueprint v4 to v5 and noticed this issue.

Guia do colaborador

Direção de pesquisa
Investigue a lógica de fechamento do componente Popover quando ocorre navegação com React Router, especialmente com carregamento lento. Verifique se o Popover escuta mudanças de rota ou se falta uma limpeza.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ReactReact RouterTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65