-
Notifications
You must be signed in to change notification settings - Fork 32.6k
Initial implementation of drag and drop api #122239
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking really good so far! I haven't gotten to test it out yet since we've just started our endgame here, but I want to try it out later this week.
This is a feature that we will want to highlight in our extension samples, and it looks like you're already testing it out as you develope there ✔ Would you also be able to make a PR so that one of the tree extension samples shows off the new API and feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few last tweaks. I intend to merge this PR on Monday (We branch for release on Friday and are mostly on taking bugfixes until then.)
This pull request has the initial work for supporting drag and drop behavior in contributed tree
I'll implement the reparent logic and refresh the tree view in next pull request
Screen.Recording.2021-04-26.at.1.41.48.PM.mov
.