fix: fix usage of useMemo (#694)

Author: mortalYoung

src/components/collapse/index.tsx

@@ -1,7 +1,6 @@
 import React, { useLayoutEffect, useMemo, useRef, useState } from 'react';
 
 import { classNames } from 'mo/common/className';
-import { isEqual } from 'lodash';
 import { HTMLElementProps, UniqueId } from 'mo/common/types';
 import { getDataAttributionsFromProps } from 'mo/common/dom';
 
@@ -16,12 +15,10 @@ import {
     collapsingClassName,
     defaultCollapseClassName,
 } from './base';
-
-import { Pane } from '../split';
 import { IActionBarItemProps } from '../actionBar';
 import { Icon } from '../icon';
 import { Toolbar } from '../toolbar';
-import SplitPane, { ResizeStratygy } from '../split/SplitPane';
+import SplitPane, { ResizeStratygy, Pane } from '../split';
 
 type RenderFunctionProps = (data: ICollapseItem) => React.ReactNode;
 export interface ICollapseItem extends HTMLElementProps {
@@ -265,11 +262,7 @@ export function Collapse({
             }
         });
 
-        const didChanged = !isEqual(res, resize);
-        if (didChanged) {
-            return res;
-        }
-        return resize;
+        return res;
     }, [sizes]);
 
     const dataAttrProps = getDataAttributionsFromProps(restProps);

src/components/split/SplitPane.tsx

@@ -5,7 +5,7 @@ import React, {
     useRef,
     useState,
 } from 'react';
-import { isEqual, throttle } from 'lodash';
+import { throttle } from 'lodash';
 import { cloneReactChildren } from 'mo/react';
 import { classNames } from 'mo/common/className';
 import { HTMLElementProps } from 'mo/common/types';
@@ -466,20 +466,13 @@ const SplitPane = ({
             limitedSizes.current[index] = registerLimitedSizes(child);
             return child.key || index;
         });
-        // improving performance
-        if (isEqual(nextKey, childrenKey)) {
-            return childrenKey;
-        }
         return nextKey;
     }, [children]);
 
     // perform the task for recalculating resizable
     const allowResize = useMemo(() => {
         if (typeof propAllowResize === 'boolean') {
             const next = new Array(childrenKey.length).fill(propAllowResize);
-            if (isEqual(next, allowResize)) {
-                return allowResize;
-            }
             return next;
         }
         const res: boolean[] = [];
@@ -490,9 +483,6 @@ const SplitPane = ({
                     : propAllowResize[index]
             );
         }
-        if (isEqual(res, allowResize)) {
-            return allowResize;
-        }
         return res;
     }, [childrenKey.length, propAllowResize]);