r3438 committed - Effects: Added support for effects on elements with right and/or botto...
Revision: 3438
Author: scott.gonzalez
Date: Tue Nov 10 08:16:52 2009
Log: Effects: Added support for effects on elements with right and/or
bottom style settings.
Fixes #4540 - Fix bug effect for IE7. In create wrapper function.
http://code.google.com/p/jquery-ui/source/detail?r=3438
Modified:
/trunk/ui/jquery.effects.core.js
=======================================
--- /trunk/ui/jquery.effects.core.js Tue Nov 10 07:55:41 2009
+++ /trunk/ui/jquery.effects.core.js Tue Nov 10 08:16:52 2009
@@ -343,11 +343,19 @@
//Transfer the positioning of the element to the wrapper
if (element.css('position') == 'static') {
wrapper.css({ position: 'relative' });
- element.css({ position: 'relative'} );
+ element.css({ position: 'relative' });
} else {
- var top = element.css('top'); if(isNaN(parseInt(top,10))) top = 'auto';
- var left = element.css('left'); if(isNaN(parseInt(left,10))) left
= 'auto';
- wrapper.css({ position: element.css('position'), top: top, left: left,
zIndex: element.css('z-index') }).show();
+ var cssProps = {
+ position: element.css('position'),
+ zIndex: element.css('z-index')
+ };
+ $.each(['top', 'left', 'bottom', 'right'], function(i, pos) {
+ cssProps[pos] = element.css(pos);
+ if (isNaN(parseInt(cssProps[pos], 10))) {
+ cssProps[pos] = 'auto';
+ }
+ });
+ wrapper.css(cssProps).show();
element.css({position: 'relative', top: 0, left: 0 });
}