palantir/blueprint

FR: Add `popoverProps` property to `TruncatedFormatProps`

Open

#6.667 geöffnet am 23. Jan. 2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (20.263 Stars) (2.167 Forks)batch import
P2Package: tableType: feature requesthelp wanted

Beschreibung

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

Contributor Guide

Tech Stack
typescriptreact
Domain
frontend
Issue Type
feature
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.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
ReactTypeScriptBlueprint components
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
Locate the `TruncatedFormat` component in the Blueprint source code (likely in `packages/core/src/components/truncated format/`). Examine the current `TruncatedFormatProps` interface and the internal rendering of the Popover. Implement the `popoverProps` prop as an optional object that spreads onto the Popover component. Ensure type safety with TypeScript. Add appropriate documentation and update any relevant tests. No existing PRs or comments clarify the implementation approach.