droppable: $.ui.ddmanager.drop now returns false in case of non-drop,…

… or the actual droppable (instead of true) in the case of a drop. draggable: implemented patch that allows revert to be a callback, receives one argument which is the droppable (implements #3175)
jquery · Sep 18, 2008 · 9c0942d14f25909bf8e3b8064965323c280f4274 · 9c0942d
1 parent c40491f
commit 9c0942d14f25909bf8e3b8064965323c280f4274
Unified Split

Showing with 2 additions and 2 deletions.

+1 −1 ui/ui.draggable.js

+1 −1 ui/ui.droppable.js
diff --git a/ui/ui.draggable.js b/ui/ui.draggable.js
@@ -309,7 +309,7 @@ $.widget("ui.draggable", $.extend({}, $.ui.mouse, {
 		if ($.ui.ddmanager && !this.options.dropBehaviour)
 			var dropped = $.ui.ddmanager.drop(this, e);		
 
-		if((this.options.revert == "invalid" && !dropped) || (this.options.revert == "valid" && dropped) || this.options.revert === true) {
+		if((this.options.revert == "invalid" && !dropped) || (this.options.revert == "valid" && dropped) || this.options.revert === true || ($.isFunction(this.options.revert) && this.options.revert.call(this.element, dropped))) {
 			var self = this;
 			$(this.helper).animate(this.originalPosition, parseInt(this.options.revertDuration, 10) || 500, function() {
 				self._propagate("stop", e);

diff --git a/ui/ui.droppable.js b/ui/ui.droppable.js
@@ -107,7 +107,7 @@ $.widget("ui.droppable", {
 		if(this.options.accept.call(this.element,(draggable.currentItem || draggable.element))) {
 			$.ui.plugin.call(this, 'drop', [e, this.ui(draggable)]);
 			this.element.triggerHandler("drop", [e, this.ui(draggable)], this.options.drop);
-			return true;
+			return this.element;
 		}
 
 		return false;