fix: handle empty or falsy column widths in FixedHolder (#1333)

Co-authored-by: Copilot <[email protected]>

Author: afc163

src/ColGroup.tsx

@@ -22,7 +22,7 @@ function ColGroup<RecordType>({ colWidths, columns, columCount }: ColGroupProps<
   for (let i = len - 1; i >= 0; i -= 1) {
     const width = colWidths[i];
     const column = columns && columns[i];
-    let additionalProps;
+    let additionalProps: Record<string, unknown>;
     let minWidth: number;
     if (column) {
       additionalProps = column[INTERNAL_COL_DEFINE];

src/FixedHolder/index.tsx

@@ -139,7 +139,13 @@ const FixedHolder = React.forwardRef<HTMLDivElement, FixedHeaderProps<any>>((pro
 
   const colGroupNode = useMemo(() => {
     // use original ColGroup if no data or no calculated column width, otherwise use calculated column width
-    if (noData || !mergedColumnWidth) {
+    // 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 (
@@ -149,7 +155,14 @@ const FixedHolder = React.forwardRef<HTMLDivElement, FixedHeaderProps<any>>((pro
         columns={flattenColumnsWithScrollbar}
       />
     );
-  }, [noData, mergedColumnWidth, colGroup, combinationScrollBarSize, columCount, flattenColumnsWithScrollbar]);
+  }, [
+    noData,
+    mergedColumnWidth,
+    colGroup,
+    combinationScrollBarSize,
+    columCount,
+    flattenColumnsWithScrollbar,
+  ]);
 
   return (
     <div

tests/__snapshots__/Table.spec.jsx.snap

@@ -216,20 +216,7 @@ LoadedCheerio {
         <table
           style="table-layout: fixed; width: 100px; min-width: 100%;"
         >
-          <colgroup>
-            <col
-              style="width: 0px;"
-            />
-            <col
-              style="width: 0px;"
-            />
-            <col
-              style="width: 0px;"
-            />
-            <col
-              style="width: 15px;"
-            />
-          </colgroup>
+          <colgroup />
           <thead
             class="rc-table-thead"
             name="my-header-wrapper"