r2855 - positionTo: storing width/height in separate vars instead of a hash (fewer bytes).

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;
        }