Skip to content

fix(explore): disable drop transition on group-by sortable items#110676

Open
JoshuaKGoldberg wants to merge 1 commit intomasterfrom
JoshuaKGoldberg/fix/group-by-drop-animation
Open

fix(explore): disable drop transition on group-by sortable items#110676
JoshuaKGoldberg wants to merge 1 commit intomasterfrom
JoshuaKGoldberg/fix/group-by-drop-animation

Conversation

@JoshuaKGoldberg
Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg commented Mar 13, 2026

Disable the dnd-kit drop transition on group-by sortable items in the explore toolbar to fix a visual animation glitch when reordering.

The state update on drop already handles visual reordering instantly. The dnd-kit drop transition conflicts with this, causing items to briefly animate from a stale position. Setting transition: null in useSortable matches the pattern already used in SortableVisualizeFieldWrapper and SortableQueryField.

I'd noticed & fixed this same kind of bug when QAing #110648. Example place where this one pops up: Explore > Traces.

Screen.Recording.2026-03-13.at.3.57.00.PM.mov

Fixes EXP-842.

Made with Cursor

Same animation glitch as the metrics toolbar — the dnd-kit drop
transition conflicts with the state update that reorders the DOM.
Matches the pattern already used in SortableVisualizeFieldWrapper
and SortableQueryField.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Made-with: Cursor
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 13, 2026
@JoshuaKGoldberg JoshuaKGoldberg changed the title fix(explore): Disable drop transition on group-by sortable items fix(explore): disable drop transition on group-by sortable items Mar 13, 2026
@linear-code
Copy link

linear-code bot commented Mar 13, 2026

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review March 13, 2026 20:09
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team as a code owner March 13, 2026 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant