r3008 committed - Position: Removed default options hash.

r3008 committed - Position: Removed default options hash.


Revision: 3008
Author: scott.gonzalez
Date: Thu Jul 30 17:15:36 2009
Log: Position: Removed default options hash.
http://code.google.com/p/jquery-ui/source/detail?r=3008
Modified:
/trunk/ui/ui.position.js
=======================================
--- /trunk/ui/ui.position.js    Thu Jul 30 07:06:31 2009
+++ /trunk/ui/ui.position.js    Thu Jul 30 17:15:36 2009
@@ -21,8 +21,6 @@
    if (!options || !options.of) {
        return _position.apply(this, arguments);
    }
-
-    options = $.extend({}, $.ui.position.defaults, options);
    var target = $(options.of),
        collision = (options.collision || 'flip').split(' '),
@@ -51,7 +49,7 @@
    // force my and at to have valid horizontal and veritcal positions
    // if a value is missing or invalid, it will be converted to center
    $.each(['my', 'at'], function() {
-        var pos = ( options[this] || "" ).split(' ');
+        var pos = (options[this] || '').split(' ');
        pos = pos.length == 1
            ? horizontalPositions.test(pos[0])
                ? pos.concat([verticalDefault])
@@ -137,7 +135,7 @@
                }));
        });
-        (options.stackfix && $.fn.stackfix && elem.stackfix());
+        (options.stackfix !== false && $.fn.stackfix && elem.stackfix());
        // the by function is passed the offset values, not the position values
        // we'll need the logic from the .offset() setter to be accessible for
        // us to calculate the position values to make the by option more useful
@@ -146,9 +144,6 @@
};
$.ui.position = {
-    defaults:{
-        stackFix: true
-    },
    fit: {
        left: function(position, data) {
            var over = position.left + data.elemWidth - $(window).width() -
$(window).scrollLeft();