it has performance issue when we drag&drop on a tree with about 500 rows open all

[btnguyen76]

I'm following the doc, everything is working fine. However if we have a tree with about 500 rows openAll, it's very slow (responsive) when we drag&drop
Do you have any suggestion on how to improve performance?
Thank you!

[minop1205]

@btnguyen76 I recognize the performance challenges.
This issue is a library issue and I will update the library soon to improve it.

At the very least, I can improve it by reducing unnecessary rendering during drag operations.

[btnguyen76]

yes, When dragging all my custom nodes are keeping re-rendered. Do you know when you are able to improve it by reducing unnecessary rendering during drag operations? Thanks @minop1205

[minop1205]

The problem is caused where the top level component in the library is re-rendered during a drag operation.

Therefore, a library modification is required to correct the problem. I am currently working on resolving this issue, but have not yet completed the process.

[JorisPannekeet]

@minop1205 Any update about this?

[kibertoad]

@minop1205 Any updates?

[jorgedanisc]

@minop1205 Are you continuing to work on this?

[yuri2peter]

Hey everyone! I've uncovered a way to drastically reduce re-renders for our CustomNode component, and in my project, it's resulted in a significant performance boost !

The key is using useFuncImmutable to make the changing handles immutable, allowing React.memo to work.

Hope this helps you out!

type CustomNodeProps = {
  node: TreeNode;
  depth: number;
  isOpen: boolean;
  onToggle: () => void;
  onClick: () => void;
  onCtrlClick: () => void;
};

function useFuncImmutable<T extends (...args: any[]) => any>(fn: T) {
  const ref = React.useRef(fn);
  ref.current = fn;
  return React.useCallback((...args: any[]) => {
    return ref.current(...args);
  }, [])  as T;
}

export const CustomNodeWrapper: React.FC<CustomNodeProps> = (props) => {
  const onToggle = useFuncImmutable(props.onToggle);
  const onClick = useFuncImmutable(props.onClick);
  const onCtrlClick = useFuncImmutable(props.onCtrlClick);
  return (
    <CustomNodeMemo
      {...props}
      {...{
        onToggle,
        onClick,
        onCtrlClick,
      }}
    />
  );
};