Skip to content

fix: fix sidebar drop indicator and actual landing spot - #10435

Open
Ali-Sab wants to merge 1 commit into
Kong:developfrom
Ali-Sab:fix/folder-item-reorder
Open

fix: fix sidebar drop indicator and actual landing spot#10435
Ali-Sab wants to merge 1 commit into
Kong:developfrom
Ali-Sab:fix/folder-item-reorder

Conversation

@Ali-Sab

@Ali-Sab Ali-Sab commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Fixes INS-2887, buggy drag and drop in sidebar. This PR will change the way the drag and drop works by dropping react-aria for a custom implementation that is well suited for our hierarchy and structure.

It adds deeper indicators that show the user exactly and precisely what happens on reorder, where the new item will go, in, around, above, below, another item or folder. It also adds an x-region, dropping an item on the left side of a folder will drop it under the folder, not inside it, dropping an item at the middle or the right of the folder's name will drop the item into the folder, the indicator will show this behaviour by including the folder's name on the drag/hover, so that users know exactly where the item will go before dropping it.

Screen.Recording.2026-08-21.at.5.49.36.PM.mov

Copilot AI lite review requested due to automatic review settings August 21, 2026 21:46
@Ali-Sab
Ali-Sab force-pushed the fix/folder-item-reorder branch from 0f68691 to 7c36869 Compare August 21, 2026 21:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Ali-Sab
Ali-Sab force-pushed the fix/folder-item-reorder branch from 7c36869 to 96e33f6 Compare August 24, 2026 14:54
@Ali-Sab
Ali-Sab force-pushed the fix/folder-item-reorder branch from 96e33f6 to ba280fc Compare August 24, 2026 14:55
@cwangsmv
cwangsmv self-requested a review August 25, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants