palantir/blueprint

Multiple re-rendering by useHotkey

Open

#5178 aperta il 17 mar 2022

Vedi su GitHub
 (4 commenti) (1 reazione) (0 assegnatari)TypeScript (2167 fork)batch import
Domain: performanceP2Package: coreType: bughelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il problema è che useHotkey causa re render durante lo scorrimento di un elenco con hotkey vuote. Innanzitutto, rivedere l'implementazione di useHotkey nella codebase di Blueprint (probabilmente in packages/core/src/components/hotkeys/). Verificare se utilizza correttamente useEffect o useMemo. Quindi, ispezionare il CodeSandbox fornito per riprodurre e identificare la causa principale. Discutere potenziali correzioni come memoizzazione o dipendenze condizionali degli effect.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Reacthooks
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60