Skip to content

Commit

Permalink
fix: resetColumns clear widthCache
Browse files Browse the repository at this point in the history
  • Loading branch information
hemengke1997 committed Jan 10, 2022
1 parent d8c91f8 commit 7dbb709
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
8 changes: 7 additions & 1 deletion src/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import { depthFirstSearch } from './utils';
import useSafeState from './utils/useSafeState';
import useLocalColumns from './utils/useLocalColumns';
import useGetDataIndexColumns, { GETKEY, ResizableUniqIdPrefix } from './utils/useGetDataIndexColumns';
import useMemoizedFn from './utils/useMemoizedFn';

export type ColumnsState = {
width: number;
Expand Down Expand Up @@ -73,7 +74,7 @@ function useTableResizableHeader<ColumnType extends ColumnOriginType<ColumnType>

const [resizableColumns, setResizableColumns] = useSafeState<ColumnType[]>([]);

const { localColumns: columns, resetColumns } = useLocalColumns({
const { localColumns: columns, resetLocalColumns } = useLocalColumns({
columnsState,
columns: columnsProp,
resizableColumns,
Expand All @@ -83,6 +84,11 @@ function useTableResizableHeader<ColumnType extends ColumnOriginType<ColumnType>

const [triggerRender, forceRender] = React.useReducer((s) => s + 1, 0);

const resetColumns = useMemoizedFn(() => {
widthCache.current = new Map();
resetLocalColumns();
});

const onMount = React.useCallback(
(id: React.Key | undefined) => (width?: number) => {
if (width) {
Expand Down
4 changes: 2 additions & 2 deletions src/utils/useLocalColumns.ts
Original file line number Diff line number Diff line change
Expand Up @@ -96,13 +96,13 @@ function useLocalColumns<T extends ColumnOriginType<T>>({
/**
* reset
*/
const resetColumns = useMemoizedFn(() => {
const resetLocalColumns = useMemoizedFn(() => {
setLocalColumns([...(columnsProp || [])]);
});

return {
localColumns: useMemo(() => localColumns, [localColumns]),
resetColumns,
resetLocalColumns,
};
}

Expand Down

0 comments on commit 7dbb709

Please sign in to comment.