r2855 - positionTo: storing width/height in separate vars instead of a hash (fewer bytes).
Author: scott.gonzalez
Date: Tue Jun 30 18:22:44 2009
New Revision: 2855
Modified:
branches/dev/positionTo/ui/ui.positionTo.js
Log:
positionTo: storing width/height in separate vars instead of a hash (fewer
bytes).
Modified: branches/dev/positionTo/ui/ui.positionTo.js
==============================================================================
--- branches/dev/positionTo/ui/ui.positionTo.js (original)
+++ branches/dev/positionTo/ui/ui.positionTo.js Tue Jun 30 18:22:44 2009
@@ -21,14 +21,11 @@
}, options);
var target = $(options.of),
- targetProps = {
- offset: target.offset(),
- width: target.outerWidth(),
- height: target.outerHeight()
- },
+ targetWidth = target.outerWidth(),
+ targetHeight = target.outerHeight(),
- offset = options.offset.replace(/px/gi, '').split(' '),
- position = targetProps.offset;
+// offset = options.offset.replace(/px/gi, '').split(' '),
+ position = target.offset();
$.each(['my', 'at'], function() {
var pos = options[this].split(' ');
@@ -48,10 +45,10 @@
case 'left':
break;
case 'right':
- position.left += targetProps.width;
+ position.left += targetWidth;
break;
default:
- position.left += targetProps.width / 2;
+ position.left += targetWidth / 2;
break;
}
@@ -59,28 +56,26 @@
case 'top':
break;
case 'bottom':
- position.top += targetProps.height;
+ position.top += targetHeight;
break;
default:
- position.top += targetProps.height / 2;
+ position.top += targetHeight / 2;
break;
}
return this.each(function() {
var elem = $(this),
- elemProps = {
- width: elem.outerWidth(),
- height: elem.outerHeight()
- };
+ elemWidth = elem.outerWidth(),
+ elemHeight = elem.outerHeight();
switch (options.my[0]) {
case 'left':
break;
case 'right':
- position.left -= elemProps.width;
+ position.left -= elemWidth;
break;
default:
- position.left -= elemProps.width / 2;
+ position.left -= elemWidth / 2;
break;
}
@@ -88,10 +83,10 @@
case 'top':
break;
case 'bottom':
- position.top -= elemProps.height;
+ position.top -= elemHeight;
break;
default:
- position.top -= elemProps.height / 2;
+ position.top -= elemHeight / 2;
break;
}