Skip to content
Permalink
Browse files

droppable: 'this' in accept should point to the DOMelement not to a j…

…Query (fixes #3955)
  • Loading branch information
Paul Bakaus
Paul Bakaus committed Jan 27, 2009
1 parent 98fc50a commit 24d450b7fe31e8f45628301f6df91badd00d71ca
Showing with 5 additions and 5 deletions.
  1. +5 −5 ui/ui.droppable.js
@@ -76,7 +76,7 @@ $.widget("ui.droppable", {
var draggable = $.ui.ddmanager.current;
if (!draggable || (draggable.currentItem || draggable.element)[0] == this.element[0]) return; // Bail if draggable and droppable are same element

if (this.options.accept.call(this.element,(draggable.currentItem || draggable.element))) {
if (this.options.accept.call(this.element[0],(draggable.currentItem || draggable.element))) {
if(this.options.hoverClass) this.element.addClass(this.options.hoverClass);
this._trigger('over', event, this.ui(draggable));
}
@@ -88,7 +88,7 @@ $.widget("ui.droppable", {
var draggable = $.ui.ddmanager.current;
if (!draggable || (draggable.currentItem || draggable.element)[0] == this.element[0]) return; // Bail if draggable and droppable are same element

if (this.options.accept.call(this.element,(draggable.currentItem || draggable.element))) {
if (this.options.accept.call(this.element[0],(draggable.currentItem || draggable.element))) {
if(this.options.hoverClass) this.element.removeClass(this.options.hoverClass);
this._trigger('out', event, this.ui(draggable));
}
@@ -109,7 +109,7 @@ $.widget("ui.droppable", {
});
if(childrenIntersection) return false;

if(this.options.accept.call(this.element,(draggable.currentItem || draggable.element))) {
if(this.options.accept.call(this.element[0],(draggable.currentItem || draggable.element))) {
if(this.options.activeClass) this.element.removeClass(this.options.activeClass);
if(this.options.hoverClass) this.element.removeClass(this.options.hoverClass);
this._trigger('drop', event, this.ui(draggable));
@@ -203,7 +203,7 @@ $.ui.ddmanager = {

droppablesLoop: for (var i = 0; i < m.length; i++) {

if(m[i].options.disabled || (t && !m[i].options.accept.call(m[i].element,(t.currentItem || t.element)))) continue; //No disabled and non-accepted
if(m[i].options.disabled || (t && !m[i].options.accept.call(m[i].element[0],(t.currentItem || t.element)))) continue; //No disabled and non-accepted
for (var j=0; j < list.length; j++) { if(list[j] == m[i].element[0]) { m[i].proportions.height = 0; continue droppablesLoop; } }; //Filter out elements in the current dragged item
m[i].visible = m[i].element.css("display") != "none"; if(!m[i].visible) continue; //If the element is not visible, continue

@@ -224,7 +224,7 @@ $.ui.ddmanager = {
if (!this.options.disabled && this.visible && $.ui.intersect(draggable, this, this.options.tolerance))
dropped = this._drop.call(this, event);

if (!this.options.disabled && this.visible && this.options.accept.call(this.element,(draggable.currentItem || draggable.element))) {
if (!this.options.disabled && this.visible && this.options.accept.call(this.element[0],(draggable.currentItem || draggable.element))) {
this.isout = 1; this.isover = 0;
this._deactivate.call(this, event);
}

0 comments on commit 24d450b

Please sign in to comment.
You can’t perform that action at this time.