Skip to content

Commit dc9f00e

Browse files
committed
chore: remove comments from scroll drag code and tests
1 parent 1fcd09d commit dc9f00e

2 files changed

Lines changed: 1 addition & 13 deletions

File tree

src/hooks/useScrollDrag.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ export default function useScrollDrag(
3838
});
3939
};
4040

41-
// 清理拖拽状态的统一函数
4241
const clearDragState = () => {
4342
mouseDownLock = false;
4443
stopScroll();

tests/scroll.test.js

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -735,7 +735,6 @@ describe('List.Scroll', () => {
735735
it('should not scroll after drop table text', () => {
736736

737737
const onScroll = jest.fn();
738-
// 这两个事件绑定在 document
739738
const onDragStart = jest.fn();
740739
const onDragEnd = jest.fn();
741740
document.addEventListener('dragstart', onDragStart);
@@ -753,50 +752,42 @@ describe('List.Scroll', () => {
753752
{({ id }) => <li className="fixed-item">{id}</li>}
754753
</List>,
755754
);
756-
// 选择第一个可见的 fixed-item 的文本内容
757755
const fixedItems = container.querySelectorAll('.fixed-item');
758-
const targetItem = fixedItems[0]; // 使用第一个可见元素
756+
const targetItem = fixedItems[0];
759757
if (targetItem) {
760758
const range = document.createRange();
761759
range.selectNodeContents(targetItem);
762760
const selection = window.getSelection();
763761
selection.removeAllRanges();
764762
selection.addRange(range);
765763
}
766-
// 选中 fixed-item 里的文本并拖拽文本到列表底部
767764
const listHolder = container.querySelector('.rc-virtual-list-holder');
768765
if (targetItem && listHolder) {
769-
// 创建选区,选中 fixed-item 的文本
770766
const range = document.createRange();
771767
range.selectNodeContents(targetItem);
772768
const selection = window.getSelection();
773769
selection.removeAllRanges();
774770
selection.addRange(range);
775771

776-
// 模拟拖拽文本
777772
fireEvent.dragStart(targetItem, { bubbles: true });
778773

779-
// 拖拽到列表底部
780774
const rect = listHolder.getBoundingClientRect();
781775
fireEvent.dragOver(listHolder, {
782776
clientY: rect.bottom + 10,
783777
bubbles: true,
784778
});
785779

786-
// 松开鼠标
787780
fireEvent.drop(listHolder, {
788781
clientY: rect.bottom + 10,
789782
bubbles: true,
790783
});
791784

792785
fireEvent.dragEnd(targetItem, { bubbles: true });
793786
}
794-
// 检查 onScroll 没有被触发
795787
expect(onScroll).not.toHaveBeenCalled();
796788
expect(onDragStart).toHaveBeenCalled();
797789
expect(onDragEnd).toHaveBeenCalled();
798790

799-
// 模拟鼠标移动到列表顶部
800791
if (listHolder) {
801792
const rect = listHolder.getBoundingClientRect();
802793
const mouseMoveEvent = new MouseEvent('mousemove', {
@@ -805,10 +796,8 @@ describe('List.Scroll', () => {
805796
});
806797
listHolder.dispatchEvent(mouseMoveEvent);
807798
}
808-
// 检查 onScroll 没有被触发
809799
expect(onScroll).not.toHaveBeenCalled();
810800

811-
// 清理事件监听器
812801
document.removeEventListener('dragstart', onDragStart);
813802
document.removeEventListener('dragend', onDragEnd);
814803
});

0 commit comments

Comments
 (0)