fix #140 empty context

[danielsussa]

Fix empty node element:
Uncaught (in promise) TypeError: Cannot set properties of null (setting '__draggable_context')
Simple error handler

const getHtmlElementFromNode = (node) => {
  const el =
      node.el || (Array.isArray(node.children) && node.children[0].el.parentNode);
  return el || {};
};

[marshallswain]

@danielsussa, thanks for the PR. Since this package and repo appears to be unmaintained, I published your changes to a new package: https://www.npmjs.com/package/@marshallswain/vuedraggable.

This project needs a lot of love. If I can find time, I'll update the tooling to use vitest, vite, and vitepress. It definitely needs a TS rewrite, too.

[marshallswain]

It looks like the changes break the onDragStart handler., which references this.context.element, but it's undefined. Not sure what's different in my setup.

[danielsussa]

hey @marshallswain , I will perform some tests with this change with my project. Good to know that you will help to put some love in there!

[danielsussa]

I just applied an NPM install using your version! Worked pretty!!
Thanks @marshallswain. I can help on further demands.

[zzliux]

@marshallswain 感谢，我换成你的包后完美解决draggable与vue3的transition-group不兼容的问题了

[Kill-pig-fly]

@marshallswain Thank you very much. However, my .list-enter-active and .list-leave-active classes are not being triggered. Only the .list-move class is being applied when removing or adding elements.

code:
tag="transition-group"
:component-data="{tag:'ul',name:'list'}"

[rendomnet]

when thiss will be merged?

[NtchPlayer]

@marshallswain Thank you very much. However, my .list-enter-active and .list-leave-active classes are not being triggered. Only the .list-move class is being applied when removing or adding elements.

code: tag="transition-group" :component-data="{tag:'ul',name:'list'}"

Yes I faced the same issue, .list-enter-active and .list-enter-from is not trigger when I add a element to my list.

[kelen-cn]

Thank you very much

[icleitoncosta]

@danielsussa, thanks for the PR. Since this package and repo appears to be unmaintained, I published your changes to a new package: https://www.npmjs.com/package/@marshallswain/vuedraggable.

This project needs a lot of love. If I can find time, I'll update the tooling to use vitest, vite, and vitepress. It definitely needs a TS rewrite, too.

Hello, I noticed that you released the project on NPM but haven't continued with it. I can help maintain it, or we can create a new repository to take over. I'm already doing this for other public projects.

Looking forward to your response. Best regards,

[marshallswain]

Search for Vue-draggable-plus. The primary language is Chinese and you can switch the docs to English. Those guys have adopted it and made it great. No need to maintain our own.