Fix to make placeholder spot exactly the same height. height() only works if there's no padding or borders on the placeholder item.

OscarGodson · OscarGodson · commit 54bb23a33688 · 2012-07-06T13:36:31.000-07:00
diff --git a/jquery.sortable.js b/jquery.sortable.js
@@ -65,7 +65,7 @@ $.fn.sortable = function(options) {
 			e.originalEvent.dataTransfer.dropEffect = 'move';
 			if (items.is(this)) {
 				if (options.forcePlaceholderSize) {
-					placeholder.height(dragging.height());
+					placeholder.height(dragging.outerHeight());
 				}
 				dragging.hide();
 				$(this)[placeholder.index() < $(this).index() ? 'after' : 'before'](placeholder);

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ $.fn.sortable = function(options) {`
`65`	`65`	`e.originalEvent.dataTransfer.dropEffect = 'move';`
`66`	`66`	`if (items.is(this)) {`
`67`	`67`	`if (options.forcePlaceholderSize) {`
`68`		`- placeholder.height(dragging.height());`
	`68`	`+ placeholder.height(dragging.outerHeight());`
`69`	`69`	`}`
`70`	`70`	`dragging.hide();`
`71`	`71`	`$(this)[placeholder.index() < $(this).index() ? 'after' : 'before'](placeholder);`