palantir/blueprint

EditableCell2 doesn't rerender to empty cell on sort.

Open

#6602 aperta il 13 dic 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
Package: tableStatus: needs proposalType: bughelp 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): "@blueprintjs/core": "^5.5.1", "@blueprintjs/table": "^5.0.20",
  • Operating System: Windows 11
  • Browser name and version: Firefox 120.0.1 (64 bit), Chrome 120.0.6099.71 (64 bit)

Code Sandbox

demo

Steps to reproduce

  1. Create some table data with some undefined values.
  2. Create a sortable Table2 whose cells are EditableCell2.
  3. Sort columns with undefined cells. When a cell value changes from some value to undefined, it does not re-render to a blank cell.

Actual behavior

EditableCell2 does not re-render when its value changes from some value to undefined.

Expected behavior

The cell re-renders into a blank cell whenver the value changes to undefined.

Possible solution

.

Guida contributor

Direzione di ricerca
Innanzitutto, individua il componente EditableCell2 nel pacchetto table di Blueprint (probabilmente in packages/table/src/components/editableCell/editableCell.tsx). Indaga come la cella renderizza il suo valore e se utilizza React.memo o shouldComponentUpdate per prevenire re render quando il valore cambia. Il problema è che l'ordinamento attiva un re render ma la cella non si aggiorna quando il valore diventa undefined. Cerca confronti che trattano undefined e una stringa precedente come uguali, forse a causa di controlli superficiali o dipendenze mancanti in useEffect.
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
Familiarity with ReactUnderstanding of Blueprint table components
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70