palantir/blueprint

EditableCell2 doesn't rerender to empty cell on sort.

Open

#6,602 建立於 2023年12月13日

在 GitHub 查看
 (1 留言) (0 反應) (0 負責人)TypeScript (20,263 star) (2,167 fork)batch import
Package: tableStatus: needs proposalType: bughelp wanted

描述

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

.

貢獻者指南

技術棧
typescriptreact
領域
frontend
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
fresh
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
mostly clear
前置要求
Familiarity with ReactUnderstanding of Blueprint table components
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
70
研究方向
First, locate the EditableCell2 component in the Blueprint table package (likely in packages/table/src/components/editableCell/editableCell.tsx). Investigate how the cell renders its value and whether it uses React.memo or shouldComponentUpdate to prevent re renders when the value changes. The issue is that sorting triggers a re render but the cell does not update when the value becomes undefined. Look for comparisons that treat undefined and a previous string as equal, perhaps due to shallow checking or missing dependency in useEffect.