tdesign-vue-next icon indicating copy to clipboard operation
tdesign-vue-next copied to clipboard

Published 20 hours ago • verified publisher icon Tencent

[Table] 拖拽排序功能异常

Open kkk48ku opened this issue 3 years ago • 4 comments
trafficstars

tdesign-vue-next 版本

0.22.0

重现链接

https://tdesign.tencent.com/vue-next/components/table#%E7%A4%BA%E4%BE%8B%E4%BA%8C%EF%BC%9A%E6%9C%89%E6%89%8B%E6%9F%84%E5%88%97%E7%9A%84%E8%A1%8C%E6%8B%96%E6%8B%BD%E6%8E%92%E5%BA%8F

重现步骤

https://user-images.githubusercontent.com/41846364/191157613-5ce887d0-8e1f-4600-8632-30e01058aa85.mov

表格拖拽排序 拖拽的速度稍微快一点 排序数据就会乱掉

期望结果

No response

实际结果

No response

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

kkk48ku avatar Sep 20 '22 02:09 kkk48ku

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

github-actions[bot] avatar Sep 20 '22 03:09 github-actions[bot]

♥️ 有劳 @realyuyanan @Louiszhai @pengYYYYY @chaishi 尽快确认问题。 确认有效后将下一步计划和可能需要的时间回复给 @kkk48ku 。

github-actions[bot] avatar Sep 20 '22 03:09 github-actions[bot]

dragInstanceTmp?.sort(lastRowList.value); 貌似是源码这里的问题 注释掉就没问题

kkk48ku avatar Sep 20 '22 03:09 kkk48ku

可以提个PR？

PengYYYYY avatar Sep 21 '22 06:09 PengYYYYY

dragInstanceTmp?.sort(lastRowList.value)

@kkk48ku 这个是为了处理受控，保证 DOM 一定和 data 一致。如果注释掉这一行，也会有个问题，就是即使 onDragSort 中没有设置新数据 data.value = params.newData; ，DOM 也依旧会变化

chaishi avatar Dec 04 '22 05:12 chaishi

针对 Vue3 特殊处理 resolved in https://github.com/Tencent/tdesign-vue-next/pull/2105 https://github.com/Tencent/tdesign-vue-next/pull/2105/commits/d1fc58302908cc86d3cce512b55dd749fa5f2f35

chaishi avatar Dec 04 '22 05:12 chaishi