Fix cellContext no longer triggering rerenders

- performance changes worked I guess!
elastic · Mar 8, 2024 · 6982d58 · 6982d58
1 parent b1ea642
commit 6982d58
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/src/components/datagrid/body/cell/data_grid_cell.test.tsx b/src/components/datagrid/body/cell/data_grid_cell.test.tsx
@@ -175,6 +175,10 @@ describe('EuiDataGridCell', () => {
             style: { top: 0, left: 0, width: 50, height: 10 },
           });
         });
+        it('cellContext', () => {
+          component.setProps({ cellContext: { someData: true } });
+          component.setProps({ cellContext: { someData: false } });
+        });
       });
 
       describe('when state changes:', () => {

diff --git a/src/components/datagrid/body/cell/data_grid_cell.tsx b/src/components/datagrid/body/cell/data_grid_cell.tsx
@@ -392,6 +392,7 @@ export class EuiDataGridCell extends Component<
       if (nextProps.style.width !== this.props.style.width) return true;
     }
 
+    if (nextProps.cellContext !== this.props.cellContext) return true;
     if (nextState.cellProps !== this.state.cellProps) return true;
     if (nextState.isFocused !== this.state.isFocused) return true;
     if (nextState.isHovered !== this.state.isHovered) return true;