palantir/blueprint

[V5] ContextMenu does not lazy render popover anymoe

Open

#6.171 geöffnet am 23. Mai 2023

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (2.167 Forks)batch import
P2Package: coreType: bughelp wanted

Repository-Metriken

Stars
 (20.263 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 43T 10h) (27 gemergte PRs in 30 T)

Beschreibung

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...

Contributor Guide

Research-Richtung
Prüfen Sie die ContextMenu Komponente, um zu verstehen, wie popoverProps gefiltert werden. Stellen Sie sicher, dass 'lazy' entweder durchgereicht oder als Standard gesetzt wird.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ReactTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60