tdesign-react icon indicating copy to clipboard operation
tdesign-react copied to clipboard

[Dialog] useDialogPosition 绑定了 getClickPosition,多次渲染没有清除,导致性能下降

Open moecasts opened this issue 1 year ago • 2 comments

tdesign-react 版本

1.5.1

重现链接

No response

重现步骤

该部分代码只添加了事件,但没有清除,导致一直累加,多次渲染后导致性能严重下降。

https://github.com/Tencent/tdesign-react/blob/1718fbef56f3e503a5b34489b1af7d4f9c59aef8/src/dialog/hooks/useDialogPosition.ts#L16C1-L18C4

image

期望结果

No response

实际结果

No response

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

下面的 useEffect 已经处理了事件绑定,这部分是不是可以删掉?

moecasts avatar Feb 21 '24 12:02 moecasts

👋 @moecasts,感谢给 TDesign 提出了 issue。 请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

github-actions[bot] avatar Feb 21 '24 12:02 github-actions[bot]

♥️ 有劳 @ldwonday @psaren @huoyuhao @honkinglin 尽快确认问题。 确认有效后将下一步计划和可能需要的时间回复给 @moecasts 。

github-actions[bot] avatar Feb 21 '24 12:02 github-actions[bot]

fixed 1.5.2

HaixingOoO avatar Feb 29 '24 08:02 HaixingOoO