Hey guys,
I feel like calling kill() on a Draggable instance should probably clean up after itself and reset the DOM element it was invoked on to it's pre-initialization state.
Currently it leaves 'user-select: text' on the inline style attribute of each element, overwriting any default user-select settings that may have been previously defined (inline or css).
See line 903 in Draggable.js
_setSelectable = function(elements, selectable) {
var i = elements.length,
e;
while (--i > -1) {
e = elements[i];
e.ondragstart = e.onselectstart = selectable ? null : _emptyFunc;
_setStyle(e, "userSelect", (selectable ? "text" : "none"));
}
}
I can of course manually clean up the style in my implementations (which is what I currently do) but I think it would be much cleaner if Draggable would keep an internal reference to any style properties it is going to modify and reset the element to that reference after the Draggable instance is killed.
Sorry for being nitpicky. Keep up the great work!
Jonathan