chore: clean up

Author: zombieJ

src/Cell/index.tsx

@@ -116,8 +116,12 @@ function Cell<RecordType>(props: CellProps<RecordType>) {
   } = props;
 
   const cellPrefixCls = `${prefixCls}-cell`;
-  const { allColumnsFixedLeft, rowHoverable } = useContext(TableContext, [
-    'allColumnsFixedLeft',
+  const {
+    // zombieJ: not used anymore?
+    // allColumnsFixedLeft,
+    rowHoverable,
+  } = useContext(TableContext, [
+    // 'allColumnsFixedLeft',
     'rowHoverable',
   ]);
 
@@ -199,12 +203,8 @@ function Cell<RecordType>(props: CellProps<RecordType>) {
       // Fixed
       [`${cellPrefixCls}-fix`]: isFixStart || isFixEnd,
       [`${cellPrefixCls}-fix-start`]: isFixStart,
-      // [`${cellPrefixCls}-fix-left-first`]: firstFixLeft && supportSticky,
-      // [`${cellPrefixCls}-fix-left-last`]: lastFixLeft && supportSticky,
       // [`${cellPrefixCls}-fix-left-all`]: lastFixLeft && allColumnsFixedLeft && supportSticky,
       [`${cellPrefixCls}-fix-end`]: isFixEnd,
-      // [`${cellPrefixCls}-fix-right-first`]: firstFixRight && supportSticky,
-      // [`${cellPrefixCls}-fix-right-last`]: lastFixRight && supportSticky,
 
       // Fixed shadow
       [`${cellPrefixCls}-fix-start-shadow`]: fixedStartShadow,

src/Table.tsx

@@ -290,7 +290,7 @@ function Table<RecordType extends DefaultRecordType>(
   const scrollX = scroll?.x;
   const [componentWidth, setComponentWidth] = React.useState(0);
 
-  const [columns, flattenColumns, flattenScrollX, hasGapFixed] = useColumns(
+  const [columns, flattenColumns, flattenScrollX] = useColumns(
     {
       ...props,
       ...expandableConfig,
@@ -777,7 +777,6 @@ function Table<RecordType extends DefaultRecordType>(
         [`${prefixCls}-fixed-header`]: fixHeader,
         /** No used but for compatible */
         [`${prefixCls}-fixed-column`]: fixColumn,
-        [`${prefixCls}-fixed-column-gapped`]: fixColumn && hasGapFixed,
         [`${prefixCls}-scroll-horizontal`]: horizonScroll,
         [`${prefixCls}-has-fix-left`]: flattenColumns[0] && flattenColumns[0].fixed,
         [`${prefixCls}-has-fix-right`]:

src/hooks/useColumns/index.tsx

@@ -152,7 +152,6 @@ function useColumns<RecordType>(
   columns: ColumnsType<RecordType>,
   flattenColumns: readonly ColumnType<RecordType>[],
   realScrollWidth: undefined | number,
-  hasGapFixed: boolean,
 ] {
   const baseColumns = React.useMemo<ColumnsType<RecordType>>(() => {
     const newColumns = columns || convertChildrenToColumns(children) || [];
@@ -275,50 +274,14 @@ function useColumns<RecordType>(
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [mergedColumns, direction, scrollWidth]);
 
-  // ========================= Gap Fixed ========================
-  // TODO: Remove this. We no need gapped check anymore
-  const hasGapFixed = React.useMemo(() => {
-    // Fixed: left, since old browser not support `findLastIndex`, we should use reverse loop
-    let lastLeftIndex = -1;
-    for (let i = flattenColumns.length - 1; i >= 0; i -= 1) {
-      const colFixed = flattenColumns[i].fixed;
-      if (colFixed === 'left' || colFixed === true) {
-        lastLeftIndex = i;
-        break;
-      }
-    }
-
-    if (lastLeftIndex >= 0) {
-      for (let i = 0; i <= lastLeftIndex; i += 1) {
-        const colFixed = flattenColumns[i].fixed;
-        if (colFixed !== 'left' && colFixed !== true) {
-          return true;
-        }
-      }
-    }
-
-    // Fixed: right
-    const firstRightIndex = flattenColumns.findIndex(({ fixed: colFixed }) => colFixed === 'right');
-    if (firstRightIndex >= 0) {
-      for (let i = firstRightIndex; i < flattenColumns.length; i += 1) {
-        const colFixed = flattenColumns[i].fixed;
-        if (colFixed !== 'right') {
-          return true;
-        }
-      }
-    }
-
-    return false;
-  }, [flattenColumns]);
-
   // ========================= FillWidth ========================
   const [filledColumns, realScrollWidth] = useWidthColumns(
     flattenColumns,
     scrollWidth,
     clientWidth,
   );
 
-  return [mergedColumns, filledColumns, realScrollWidth, hasGapFixed];
+  return [mergedColumns, filledColumns, realScrollWidth];
 }
 
 export default useColumns;