Workfront / glide-data-grid / 31793567737

89%

main: 89%

push

github

fix: reset isActivelyDraggingHeader and scrollDir on native dragend

Header column reordering uses native HTML5 drag-and-drop. isActivelyDraggingHeader
is set true on mousedown for header cells, but was only ever cleared in onMouseUp.
Native DnD release delivers dragend, not mouseup, to the drag source, so the ref
stayed stuck true after any header drag. Every subsequent mousemove then re-armed
scrollDir from the live cursor position (see onMouseMoveImpl), driving the
autoscroll RAF loop indefinitely with no mouse button held, until the user
produced a real mouseup via an unrelated click.

Reset both isActivelyDraggingHeader and scrollDir in onDragEnd, which already
fires unconditionally on the native dragend event.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

3071 of 3788 branches covered (81.07%)

Branch coverage included in aggregate %.

0 of 2 new or added lines in 1 file covered. (0.0%)

18608 of 20506 relevant lines covered (90.74%)

3253.38 hits per line