Skip to content

Commit

Permalink
Effects: Making sure outerHeight and outerWidth make it through to th…
Browse files Browse the repository at this point in the history
…e .size() implementation in all cases - Fixes IE8 effects failure

(cherry picked from commit a217bd3)
  • Loading branch information
gnarf committed Nov 22, 2012
1 parent a0098ea commit 188259e
Showing 1 changed file with 24 additions and 7 deletions.
31 changes: 24 additions & 7 deletions ui/jquery.ui.effect-scale.js
Expand Up @@ -21,7 +21,9 @@ $.effects.effect.puff = function( o, done ) {
factor = percent / 100,
original = {
height: elem.height(),
width: elem.width()
width: elem.width(),
outerHeight: elem.outerHeight(),
outerWidth: elem.outerWidth()
};

$.extend( o, {
Expand All @@ -35,7 +37,9 @@ $.effects.effect.puff = function( o, done ) {
original :
{
height: original.height * factor,
width: original.width * factor
width: original.width * factor,
outerHeight: original.outerHeight * factor,
outerWidth: original.outerWidth * factor
}
});

Expand Down Expand Up @@ -74,7 +78,12 @@ $.effects.effect.scale = function( o, done ) {
options.restore = true;
}

options.from = o.from || ( mode === "show" ? { height: 0, width: 0 } : original );
options.from = o.from || ( mode === "show" ? {
height: 0,
width: 0,
outerHeight: 0,
outerWidth: 0
} : original );
options.to = {
height: original.height * factor.y,
width: original.width * factor.x,
Expand Down Expand Up @@ -124,7 +133,9 @@ $.effects.effect.size = function( o, done ) {
props = restore ? props0 : props1,
zero = {
height: 0,
width: 0
width: 0,
outerHeight: 0,
outerWidth: 0
};

if ( mode === "show" ) {
Expand Down Expand Up @@ -213,19 +224,25 @@ $.effects.effect.size = function( o, done ) {
var child = $( this ),
c_original = {
height: child.height(),
width: child.width()
width: child.width(),
outerHeight: child.outerHeight(),
outerWidth: child.outerWidth()
};
if (restore) {
$.effects.save(child, props2);
}

child.from = {
height: c_original.height * factor.from.y,
width: c_original.width * factor.from.x
width: c_original.width * factor.from.x,
outerHeight: c_original.outerHeight * factor.from.y,
outerWidth: c_original.outerWidth * factor.from.x
};
child.to = {
height: c_original.height * factor.to.y,
width: c_original.width * factor.to.x
width: c_original.width * factor.to.x,
outerHeight: c_original.height * factor.to.y,
outerWidth: c_original.width * factor.to.x
};

// Vertical props scaling
Expand Down

0 comments on commit 188259e

Please sign in to comment.