<?xml version="1.0" encoding="UTF-8"?>
<commit>
  <added type="array"/>
  <modified type="array">
    <modified>
      <diff>@@ -331,36 +331,48 @@ $.extend($.effects, {
 	// Wraps the element around a wrapper that copies position properties
 	createWrapper: function(element) {
 
-		//if the element is already wrapped, return it
-		if (element.parent().is('.ui-effects-wrapper'))
+		// if the element is already wrapped, return it
+		if (element.parent().is('.ui-effects-wrapper')) {
 			return element.parent();
+		}
+
+		// wrap the element
+		var props = {
+				width: element.outerWidth(true),
+				height: element.outerHeight(true),
+				'float': element.css('float')
+			},
+			wrapper = $('&lt;div&gt;&lt;/div&gt;')
+				.addClass('ui-effects-wrapper')
+				.css({
+					fontSize: '100%',
+					background: 'transparent',
+					border: 'none',
+					margin: 0,
+					padding: 0
+				});
 
-		//Cache width,height and float properties of the element, and create a wrapper around it
-		var props = { width: element.outerWidth(true), height: element.outerHeight(true), 'float': element.css('float') };
-		element.wrap('&lt;div class=&quot;ui-effects-wrapper&quot; style=&quot;font-size:100%;background:transparent;border:none;margin:0;padding:0&quot;&gt;&lt;/div&gt;');
-		var wrapper = element.parent();
+		element.wrap(wrapper);
 
-		//Transfer the positioning of the element to the wrapper
+		// transfer positioning properties to the wrapper
 		if (element.css('position') == 'static') {
 			wrapper.css({ position: 'relative' });
 			element.css({ position: 'relative' });
 		} else {
-			var cssProps = {
+			$.extend(props, {
 				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';
+				props[pos] = element.css(pos);
+				if (isNaN(parseInt(props[pos], 10))) {
+					props[pos] = 'auto';
 				}
 			});
-			wrapper.css(cssProps).show();
 			element.css({position: 'relative', top: 0, left: 0 });
 		}
 
-		wrapper.css(props);
-		return wrapper;
+		return wrapper.css(props).show();
 	},
 
 	removeWrapper: function(element) {</diff>
      <filename>ui/jquery.effects.core.js</filename>
    </modified>
  </modified>
  <removed type="array"/>
  <parents type="array">
    <parent>
      <id>22d86e8875a784ff93bce29da4c22a1eb411ed65</id>
    </parent>
  </parents>
  <author>
    <name>scott.gonzalez</name>
    <email>scott.gonzalez@a1defee3-d24d-0410-b1e3-3171fe540af7</email>
  </author>
  <url>http://github.com/JackDanger/jquery-ui/commit/9768fd7ccb1597305dcaea0b721ddf9c4406091c</url>
  <id>9768fd7ccb1597305dcaea0b721ddf9c4406091c</id>
  <committed-date>2009-11-10T08:35:55-08:00</committed-date>
  <authored-date>2009-11-10T08:35:55-08:00</authored-date>
  <message>Effects: Cleaned up createWrapper().

git-svn-id: http://jquery-ui.googlecode.com/svn/trunk@3439 a1defee3-d24d-0410-b1e3-3171fe540af7</message>
  <tree>2a2cfeccd358dad30b50f52cbf4936197da36bdb</tree>
  <committer>
    <name>scott.gonzalez</name>
    <email>scott.gonzalez@a1defee3-d24d-0410-b1e3-3171fe540af7</email>
  </committer>
</commit>
