Jeff Robbins

Results 51 comments of Jeff Robbins

MIT is a little more flexible than GPL. What are the other options? Apache?

Sounds like this is a solved problem. Closing. Please reopen if I'm mistaken.

This is great! And I've updated [the CodePen example](https://codepen.io/jjeff/pen/rNLEZGL) with your patch. A couple of things though… and I'm not sure if they're existing bugs or issues with your patch:...

Updated the [Code Pen](https://codepen.io/jjeff/pen/rNLEZGL) again. The `multiDragKey` functionality is still not acting like I'd expect. According to [the docs](https://github.com/SortableJS/Sortable/tree/master/plugins/MultiDrag#multidragkey-option): The multiDragKey is "a custom keyboard key may be designated as...

I've been testing and I'm sometimes getting an error saying: ```Error Unhandled Exception TypeError: Cannot read properties of null (reading 'Sortable1660768120063') at t.dragOver ``` …which appears to be happening in...

Okay another approach: ```javascript dragOver(…) { … const toSortable = target.parentNode && target.parentNode[expando]; // ^^^^^^^^^^^^^^^^^^^^ ``` This seems to solve the problem.

Great! Thanks, @maxbol. This makes `multiDrag` a lot more useful for me. I hope this gets into Sortable core soon so others can benefit.

I've updated [the CodePen example](https://codepen.io/jjeff/pen/rNLEZGL) with the latest version of this PR. Works great! Closes https://github.com/SortableJS/Sortable/issues/2173

Nudge to @owen-m1 and @RubaXa. It would be great to get this into the distribution! (along with https://github.com/SortableJS/Sortable/pull/2193) 🙂

> Multi-select another item from another list will not deselect the current ones. This is being worked on [over here](https://github.com/SortableJS/Sortable/issues/2173) with a [pull request here](https://github.com/SortableJS/Sortable/pull/2181).