[EuiDataGrid] Fix timeout clean-up when component unmounts (#7534)

Co-authored-by: Marco Antonio Ghiani <[email protected]>
elastic · Feb 20, 2024 · d74601b · d74601b
1 parent 47ab766
commit d74601b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/changelogs/upcoming/7534.md b/changelogs/upcoming/7534.md
@@ -0,0 +1,3 @@
+**Bug fixes**
+
+- Fixed `EuiDataGrid` component to clean up timer from side effect on unmount
diff --git a/src/components/datagrid/utils/grid_height_width.ts b/src/components/datagrid/utils/grid_height_width.ts
@@ -183,12 +183,14 @@ export const useVirtualizeContainerWidth = (
 
   useEffect(() => {
     // wait for layout to settle, then measure virtualize container
-    setTimeout(() => {
+    const timerId = setTimeout(() => {
       if (virtualizeContainer?.clientWidth) {
         const containerWidth = virtualizeContainer.clientWidth;
         setVirtualizeContainerWidth(containerWidth);
       }
     }, 100);
+
+    return () => clearTimeout(timerId);
   }, [pageSize, virtualizeContainer]);
 
   // Use clientWidth of the virtualization container to take scroll bar into account