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 /ui | |
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.
Diffstat (limited to 'ui')
-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)); }; } }; } |