fix prevHeight

Author: Lukas742

packages/main/src/components/AnalyticalTable/index.tsx

@@ -392,11 +392,11 @@ const AnalyticalTable = forwardRef<AnalyticalTableDomRef, AnalyticalTablePropTyp
     tableInstance(tableInstanceRef.current);
   }
 
-  const titleBarRef = useRef(null);
-  const extensionRef = useRef(null);
-  const headerRef = useRef(null);
-
-  const extensionsHeight = getCombinedElementsHeight(0, titleBarRef, extensionRef, headerRef);
+  const prevExtensionsHeight = useRef<number>(0);
+  const titleBarRef = useRef<HTMLElement>(null);
+  const extensionRef = useRef<HTMLElement>(null);
+  const headerRef = useRef<HTMLElement>(null);
+  const extensionsHeight = getCombinedElementsHeight(prevExtensionsHeight, titleBarRef, extensionRef, headerRef);
 
   const internalRowHeight = getRowHeight(rowHeight, tableRef);
   const internalHeaderRowHeight = headerRowHeight ?? internalRowHeight;

packages/main/src/components/AnalyticalTable/util/index.ts

@@ -177,18 +177,23 @@ export function getLeafHeaders(header) {
   return leafHeaders;
 }
 
-export const getCombinedElementsHeight = (prevHeight: number, ...refs: RefObject<HTMLElement>[]): number => {
+export const getCombinedElementsHeight = (
+  prevHeightRef: RefObject<number>,
+  ...refs: RefObject<HTMLElement>[]
+): number => {
+  const prevHeight = prevHeightRef.current;
   let height = 0;
 
   for (const ref of refs) {
     const el = ref.current;
     if (!el) {
       continue;
     }
-    const elementHeight = el.offsetHeight;
-    height += elementHeight;
+    height += el.offsetHeight;
   }
-
   // Math.abs is required, because of layout thrashing (rounding errors)
-  return Math.abs(prevHeight - height) > 1 ? height : prevHeight;
+  const updatedHeight = Math.abs(prevHeight - height) > 1 ? height : prevHeight;
+  prevHeightRef.current = updatedHeight;
+
+  return updatedHeight;
 };