]> source.dussan.org Git - jquery-ui.git/commitdiff
sortable: fixed issue with first time append of the placeholder, now also the origina...
authorPaul Bakaus <paul.bakaus@googlemail.com>
Fri, 17 Oct 2008 08:13:14 +0000 (08:13 +0000)
committerPaul Bakaus <paul.bakaus@googlemail.com>
Fri, 17 Oct 2008 08:13:14 +0000 (08:13 +0000)
ui/ui.sortable.js

index 6c81c2e940636ef6785077297ff419c969fca33a..5fa9e4f3bc254174b40408c9072dc72675b1e975 100644 (file)
@@ -297,7 +297,10 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, {
                        var className = o.placeholder;
                        o.placeholder = {
                                element: function() {
-                                       var el = $(document.createElement(self.currentItem[0].nodeName)).addClass(className || "ui-sortable-placeholder")[0];
+                                       
+                                       var el = $(document.createElement(self.currentItem[0].nodeName))
+                                               .addClass(className || self.currentItem[0].className+" ui-sortable-placeholder")
+                                               .removeClass('ui-sortable-helper')[0];
                                        
                                        if(!className) {
                                                el.style.visibility = "hidden";
@@ -316,10 +319,15 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, {
                        };
                }
                
-               self.placeholder = $(o.placeholder.element.call(self.element, self.currentItem))
-               self.currentItem.parent()[0].appendChild(self.placeholder[0]);
-               self.placeholder[0].parentNode.insertBefore(self.placeholder[0], self.currentItem[0]);
+               //Create the placeholder
+               self.placeholder = $(o.placeholder.element.call(self.element, self.currentItem));
+               
+               //Append it after the actual current item
+               self.currentItem.after(self.placeholder);
+               
+               //Update the size of the placeholder (TODO: Logic to fuzzy, see line 316/317)
                o.placeholder.update(self, self.placeholder);
+               
        },
        
        _contactContainers: function(e) {
@@ -442,7 +450,7 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, {
                
                this.offsetParent = this.helper.offsetParent();                                                                                                 //Get the offsetParent and cache its position
                var po = this.offsetParent.offset();                    
-               
+       
                this.offsetParentBorders = {
                        top: (parseInt(this.offsetParent.css("borderTopWidth"),10) || 0),
                        left: (parseInt(this.offsetParent.css("borderLeftWidth"),10) || 0)