palantir/blueprint

[V5] ContextMenu does not lazy render popover anymoe

Open

#6171 aperta il 23 mag 2023

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

Metriche repository

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

Descrizione

Environment

  • Package version(s): 5.0.0-alpha.3
  • Operating System: MacOS
  • Browser name and version: Chrome

Steps to reproduce

  • We're migrating ContextMenu2 --> ContextMenu, but see some degrade in performance for some of our table.

Actual behavior

  • Lots of portal divs are created in advance, this is not happening with ContextMenu2, it used to be lazy, 2023-05-24-001917_611x460_scrot

  • I am trying to pass lazy: true to popoverProps but TypeScript complaining, apparently because we only pick a handful of props.

Expected behavior

  • lazy should be the default, like it was with ContextMenu2

Possible solution

I'm not too sure this time...

Guida contributor

Direzione di ricerca
Ispeziona il componente ContextMenu per capire come vengono filtrate le popoverProps. Assicurati che 'lazy' venga passato o impostato come predefinito.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
ReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60