Skip to content
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

MultiDrag: Suggest to add oldIndices and newIndices in eventOptions #1527

Closed
runninghare opened this issue Jun 2, 2019 · 2 comments
Closed
Labels
feature MultiDrag Relating to the MultiDrag plugin

Comments

@runninghare
Copy link

At the moment MultiDrag plugin only added 2 extra eventOptions, which are very useful:

  • items: an array of items being dragged
  • clones: clones of the items

However if we take a look at the original properties, we would find oldIndex and newIndex are very important as well, especially for those data binding frameworks like Angular and React. However there are no counterparts in the multiDrag version.

At present I only tested angular-sortablejs which does not work with multidrag.

Suggested actions:

  • Add 2 new properties in eventOptions: oldIndices and newIndices which allow data-binding framework to track where the dragged items come from and where they go to.
@owen-m1
Copy link
Member

owen-m1 commented Jun 3, 2019

Done in commit 87c9a06.
I am only going to rebuild the scripts at release, so you will have to clone the repo and build it yourself if you want these options ASAP.

@owen-m1 owen-m1 closed this as completed Jun 3, 2019
@runninghare
Copy link
Author

@owen-m1 Thanks for your quick updates.

Runi-c pushed a commit to Tupperbox/Sortable that referenced this issue Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature MultiDrag Relating to the MultiDrag plugin
Projects
None yet
Development

No branches or pull requests

2 participants