diff options
| author | Scott González <scott.gonzalez@gmail.com> | 2012-04-30 10:39:06 -0400 |
|---|---|---|
| committer | Scott González <scott.gonzalez@gmail.com> | 2012-04-30 10:39:06 -0400 |
| commit | 21df5c27e312c6e5fda66e602abefa51c484d676 (patch) | |
| tree | ddb9b886e97278b484bba092bc83b6501989fafc | |
| parent | b38dd26da0a7297941839195ff75885d6ecc67d0 (diff) | |
| download | jquery-ui-21df5c27e312c6e5fda66e602abefa51c484d676.tar.gz jquery-ui-21df5c27e312c6e5fda66e602abefa51c484d676.zip | |
Revert "Sortable: Always set placeholder size when necessary. Fixes #8262 - jQuery UI 1.8.19 don't respect forcePlaceholderSize."
This reverts commit 4f19289c10e733a07cf60435c2e5df4ed2e23d37.
| -rw-r--r-- | ui/jquery.ui.sortable.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ui/jquery.ui.sortable.js b/ui/jquery.ui.sortable.js index 3378192c9..a6903b263 100644 --- a/ui/jquery.ui.sortable.js +++ b/ui/jquery.ui.sortable.js @@ -673,8 +673,9 @@ $.widget("ui.sortable", $.ui.mouse, { // 2. The option 'forcePlaceholderSize can be enabled to force it even if a class name is specified if(className && !o.forcePlaceholderSize) return; - p.height(self.currentItem.height()); - p.width(self.currentItem.width()); + //If the element doesn't have a actual height by itself (without styles coming from a stylesheet), it receives the inline height from the dragged item + if(!p.height()) { p.height(self.currentItem.innerHeight() - parseInt(self.currentItem.css('paddingTop')||0, 10) - parseInt(self.currentItem.css('paddingBottom')||0, 10)); }; + if(!p.width()) { p.width(self.currentItem.innerWidth() - parseInt(self.currentItem.css('paddingLeft')||0, 10) - parseInt(self.currentItem.css('paddingRight')||0, 10)); }; } }; } |