Minor inefficiencies in Droppable
These aren't a big deal, but they'll save a few bytes.
In Droppable _init, there's the line
var o = this.options, accept = o.accept;
but o is never used and accept is only used once; everywhere else it's
this.options.accept.
Change line 23 from
this.options.accept = this.options.accept && $.isFunction
(this.options.accept) ? this.options.accept : function(d) {
to
o.accept = $.isFunction(accept) ? accept : function(d) {
($.isFunction() correctly returns false for a false-y argument, so the
"accept &&" guard is unnecessary)
Also, further down, this.options.scope can be replaced by o.scope
And
(this.options.addClasses && this.element.addClass("ui-droppable"));
is a funny idiom. Why not the clearer
if (o.addClasses) this.element.addClass("ui-droppable"));
Danny