Skip to content

Conversation

@manixate
Copy link

@manixate manixate commented May 12, 2018

Fixes: #48929

The approach here is to prevent scrollbar to be not considered a drop target when searching on dragover or dragleave. This behaviour is similar to how MacOS handles drag and drop while hovering on the scrollbar in Finder.

Another approach is basically to add drop event listener on the domNode instead of wrapper which will allow the user to drop any item on the scrollbar but the file will be dropped on the top level of the folder structure instead of in the scrolled position.

Let me know if you want another approach instead of the one in this PR.

@joaomoreno joaomoreno added this to the Backlog milestone May 28, 2018
@joaomoreno joaomoreno added the tree-widget Tree widget issues label May 28, 2018
@joaomoreno joaomoreno modified the milestones: Backlog, September 2018 Sep 11, 2018
@joaomoreno joaomoreno merged commit e9ed38c into microsoft:master Sep 11, 2018
@joaomoreno
Copy link
Member

Thanks! 🍻

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tree-widget Tree widget issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Drag and drop onto scrollbar - explorer stays highlighted

2 participants