Skip to content
Permalink
Browse files

Transfer effect: Use innerWidth/Height instead of outerWidth/Height a…

…nd then subtracting border. Fixed #4153 - Transfer throws 'Invalid Argument' in IE7 without CSS border property.
  • Loading branch information...
scottgonzalez committed Feb 17, 2009
1 parent eb5858f commit 49b76873ad4c46fe2c1636b0ad711b147b34e8b8
Showing with 4 additions and 4 deletions.
  1. +4 −4 ui/effects.transfer.js
@@ -31,8 +31,8 @@ $.effects.transfer = function(o) {
transfer.css({
top: position.top,
left: position.left,
height: el.outerHeight() - parseInt(transfer.css('borderTopWidth'),10) - parseInt(transfer.css('borderBottomWidth'),10),
width: el.outerWidth() - parseInt(transfer.css('borderLeftWidth'),10) - parseInt(transfer.css('borderRightWidth'),10),
height: el.innerHeight(),
width: el.innerWidth(),
position: 'absolute'
});

@@ -41,8 +41,8 @@ $.effects.transfer = function(o) {
animation = {
top: position.top,
left: position.left,
height: target.outerHeight() - parseInt(transfer.css('borderTopWidth'),10) - parseInt(transfer.css('borderBottomWidth'),10),
width: target.outerWidth() - parseInt(transfer.css('borderLeftWidth'),10) - parseInt(transfer.css('borderRightWidth'),10)
height: target.innerHeight(),
width: target.innerWidth()
};

// Animate

0 comments on commit 49b7687

Please sign in to comment.
You can’t perform that action at this time.