palantir/blueprint

Multiple re-rendering by useHotkey

Open

#5.178 aberto em 17 de mar. de 2022

Ver no GitHub
 (4 comments) (1 reaction) (0 assignees)TypeScript (2.167 forks)batch import
Domain: performanceP2Package: 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

Inside List components (Row) have empty hotkeys. On Scrolling list - rendering Row causes App with use useHotkey to re-render.

Environment

  • Package version(s): @blueprintjs/core v4.0.0-rc.0, react-virtualized v9.22.3
  • Operating System: Mac os 12.2.1
  • Browser name and version: Chrome 99.0.4844.51

Code Sandbox

Code sandbox

Steps to reproduce

Loom - https://www.loom.com/share/f6ad7a40cdd94bc08a2f36f7ac23a5db

Actual behavior

Component is re-rendering when scrolling List

Expected behavior

Component doesn't re-rendering when scrolling List

Guia do colaborador

Direção de pesquisa
O problema é que useHotkey causa re renderizações ao rolar uma lista com hotkeys vazias. Primeiro, revise a implementação de useHotkey na base de código do Blueprint (provavelmente em packages/core/src/components/hotkeys/). Verifique se ela usa useEffect ou useMemo corretamente. Em seguida, inspecione o CodeSandbox fornecido para reproduzir e identificar a causa raiz. Discuta possíveis correções, como memoização ou dependências condicionais de efeito.
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.
2
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
Reacthooks
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60