palantir/blueprint

FR: Add `popoverProps` property to `TruncatedFormatProps`

Open

#6667 aperta il 23 gen 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
P2Package: tableType: feature requesthelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 24g 18h) (17 PR mergiate in 30 g)

Descrizione

Environment

  • Package version(s): ^5.8.2
  • Browser and OS versions: Chrome 120.0.6099.234 & MacOS 14.2.1

Feature request

Add popoverProps property to TruncatedFormatProps

Examples

Currently, when using JSONFormat with detectTruncation=true and showPopover=true, the popover has some confusing behaviour. Would be nice to have options to configure the popover's behaviour (eg. setting interactionKind to HOVER_TARGET_ONLY to simply avoid these interactions)

  • Right clicking on the popover makes it look like you're copying the popover text but it's actually selecting the cell underneath

  • Highlighting some popover text and then cmd+c looks like you're copy/pasting the highlighted substring, but it's actually copy/pasting the entire cell

Guida contributor

Direzione di ricerca
Esamina l'implementazione del componente TruncatedFormat per capire come è attualmente configurato il popover. Guarda altri componenti Blueprint che accettano popoverProps (es. Tooltip) come riferimento. Poi aggiungi una prop popoverProps a TruncatedFormatProps e passala al componente Popover sottostante.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Funzionalità
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
GitReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60