fix: avoiding closure problems

avoiding closure problems

Author: li-jia-nan

docs/examples/measureRowRender.tsx

@@ -31,9 +31,14 @@ const data = [
 ];
 
 // 自定义 MeasureRow 渲染，隐藏弹层内容
-const measureRowRender: TableProps['measureRowRender'] = measureRow => (
-  <div style={{ display: 'none' }}>{measureRow}</div>
-);
+const measureRowRender: TableProps['measureRowRender'] = measureRow => {
+  if (React.isValidElement(measureRow)) {
+    return React.cloneElement<any>(measureRow, {
+      style: { ...(measureRow.props as any).style, display: 'none' },
+    });
+  }
+  return measureRow;
+};
 
 // 示例：使用 measureRowRender 来隐藏 MeasureRow 中的弹层
 const MeasureRowRenderExample: React.FC = () => {

src/FixedHolder/index.tsx

@@ -157,31 +157,13 @@ const FixedHolder = React.forwardRef<HTMLDivElement, FixedHeaderProps<any>>((pro
 
   const mergedColumnWidth = useColumnWidth(colWidths, columCount);
 
-  const colGroupNode = useMemo(() => {
+  const isColGroupEmpty = useMemo<boolean>(() => {
     // use original ColGroup if no data or no calculated column width, otherwise use calculated column width
     // Return original colGroup if no data, or mergedColumnWidth is empty, or all widths are falsy
-    if (
-      noData ||
-      !mergedColumnWidth ||
-      mergedColumnWidth.length === 0 ||
-      mergedColumnWidth.every(width => !width)
-    ) {
-      return ColGroup;
-    }
-    return (
-      <ColGroup
-        colWidths={[...mergedColumnWidth, combinationScrollBarSize]}
-        columCount={columCount + 1}
-        columns={flattenColumnsWithScrollbar}
-      />
-    );
-  }, [
-    noData,
-    mergedColumnWidth,
-    combinationScrollBarSize,
-    columCount,
-    flattenColumnsWithScrollbar,
-  ]);
+    const noWidth =
+      !mergedColumnWidth || !mergedColumnWidth.length || mergedColumnWidth.every(w => !w);
+    return noData || noWidth;
+  }, [noData, mergedColumnWidth]);
 
   return (
     <div
@@ -203,7 +185,13 @@ const FixedHolder = React.forwardRef<HTMLDivElement, FixedHeaderProps<any>>((pro
           width: scrollX,
         }}
       >
-        {colGroupNode}
+        {isColGroupEmpty ? null : (
+          <ColGroup
+            colWidths={[...mergedColumnWidth, combinationScrollBarSize]}
+            columCount={columCount + 1}
+            columns={flattenColumnsWithScrollbar}
+          />
+        )}
         {children({
           ...restProps,
           stickyOffsets: headerStickyOffsets,