palantir/blueprint

`eachUniqueFullRow` crashes with header row shift-click

Chiusa

#6383 aperta il 15 set 2023

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
P2Package: tableType: bughelp wanted

Metriche repository

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

Descrizione

Environment

  • Package version(s): 5.3.0
  • Operating System: MacOS
  • Browser name and version: Chrome 117

Steps to reproduce

  1. Pass a region of { row: [1, Infinity] } to eachUniqueFullRow (this can be caused for example by shift-clicking on a row header cell + using eachUniqueFullRow in the onSelect.

Actual behavior

Spins until tab crashes

Expected behavior

Short circuit or error

Guida contributor

Direzione di ricerca
Indaga la funzione `eachUniqueFullRow` nel codice sorgente della tabella Blueprint, probabilmente negli strumenti di selezione (es. src/components/table/). Esamina come gestisce gli intervalli di righe e in particolare quando viene passato `Infinity`. Considera di aggiungere una protezione per verificare `Infinity` o di limitare l'intervallo di righe al numero effettivo di righe. Controlla i test esistenti per casi limite simili.
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.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Basic understanding of Blueprint table componentFamiliarity with the issue's reproduction steps
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45