wrap handleChange with useCallback

EyeSeeTea · Jul 10, 2024 · c539623 · c539623
1 parent 10bb633
commit c539623
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/src/webapp/components/table/Cell.tsx b/src/webapp/components/table/Cell.tsx
@@ -1,4 +1,4 @@
-import React from "react";
+import React, { useCallback } from "react";
 import { TableCell, Link } from "@material-ui/core";
 import styled from "styled-components";
 import { Selector } from "../selector/Selector";
@@ -16,10 +16,13 @@ type CellProps = {
 export const Cell: React.FC<CellProps> = React.memo(
     ({ value, rowIndex, column, onChange = noop }) => {
         const [selectorValue, setSelectorValue] = React.useState<string>(value);
-        const handleChange = (value: string) => {
-            setSelectorValue(value);
-            onChange(value, rowIndex, column.value);
-        };
+        const handleChange = useCallback(
+            (value: string) => {
+                setSelectorValue(value);
+                onChange(value, rowIndex, column.value);
+            },
+            [onChange, rowIndex, column.value]
+        );
 
         switch (column.type) {
             case "link":