avoiding closure problems

docs/examples/measureRowRender.tsx

@@ -30,7 +30,9 @@ const data = [
   { key: 3, name: 'Jime', age: 35 },
 ];
 
-// 自定义 MeasureRow 渲染，隐藏弹层内容
+// 注意，这个 measureRow 实际上是一个 <tr> 标签
+// 按照 html 规范，tr 的父元素必须是 table、thead、tbody、tfoot 标签，子元素必须是 th、td 标签
+// 因此这里我们用一个 div 包裹是不对的，在控制台中会报错
 const measureRowRender: TableProps['measureRowRender'] = measureRow => (
   <div style={{ display: 'none' }}>{measureRow}</div>
 );

src/FixedHolder/index.tsx

@@ -157,31 +157,16 @@ const FixedHolder = React.forwardRef<HTMLDivElement, FixedHeaderProps<any>>((pro
 
   const mergedColumnWidth = useColumnWidth(colWidths, columCount);
 
-  const colGroupNode = useMemo(() => {
+  const mayBeEmpty = 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 (
+    return (
       noData ||
       !mergedColumnWidth ||
-      mergedColumnWidth.length === 0 ||
+      !mergedColumnWidth.length ||
       mergedColumnWidth.every(width => !width)
-    ) {
-      return ColGroup;
-    }
-    return (
-      <ColGroup
-        colWidths={[...mergedColumnWidth, combinationScrollBarSize]}
-        columCount={columCount + 1}
-        columns={flattenColumnsWithScrollbar}
-      />
     );
-  }, [
-    noData,
-    mergedColumnWidth,
-    combinationScrollBarSize,
-    columCount,
-    flattenColumnsWithScrollbar,
-  ]);
+  }, [noData, mergedColumnWidth]);
 
   return (
     <div
@@ -203,7 +188,13 @@ const FixedHolder = React.forwardRef<HTMLDivElement, FixedHeaderProps<any>>((pro
           width: scrollX,
         }}
       >
-        {colGroupNode}
+        {mayBeEmpty ? null : (
+          <ColGroup
+            colWidths={[...mergedColumnWidth, combinationScrollBarSize]}
+            columCount={columCount + 1}
+            columns={flattenColumnsWithScrollbar}
+          />
+        )}
         {children({
           ...restProps,
           stickyOffsets: headerStickyOffsets,