-
Notifications
You must be signed in to change notification settings - Fork 128
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
Dual list does not work with getter and setters #77
Comments
It seems like the behavior is different for changing sort in the list and when moving things across lists. For changing order I can find https://github.com/Alfred-Skyblue/vue-draggable-plus/blob/main/src/useDraggable.ts#L175 But for the |
Noticed this too. Was extremely happy seeing someone took up the torch to keep vue3 up to date with sortablejs. However, using computed wrapper does not work with this package :/ This is how i've done it using the outdated vuedraggable package which i hoped would still work for this but it doesnt..
the setters are not triggered when moving from list A to list B :/ I could add the data into 2 seperate ref<[]> objects but that requires more boilerplate to ensure they get populated when the api promise returns etc.. |
+1 |
I fixed this back then for myself by forking, published here: https://www.npmjs.com/package/vue-draggable-plus-plus But beware, I may have broken other functionality while fixing it for my usecase. (commits: https://github.com/MartinMalinda/vue-draggable-plus/commits/main/) |
Minimal repro:
https://codesandbox.io/p/sandbox/vue-draggable-plus-dual-list-setters-5wl5qm
If you drag items as part of one list, it works well. Once you try to drag across lists, the setters do not get called.
This worked for me before I switched the original package (vue draggable next) for this one. So something must have broken it after the fork.
The text was updated successfully, but these errors were encountered: