github/jquery-ui
1
1
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2026-05-13 03:01:39 -04:00
Code Issues Packages Projects Releases Wiki Activity

Position: Mini refactoring, avoids proxying using callback multiple times if more then one element is positioned

Browse Source
This commit is contained in:
Jörn Zaefferer
2012-04-11 00:19:26 +02:00
parent c0cceee8e4
commit 623e8e68b4
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
ui/jquery.ui.position.js vendored
View File

@@ -170,7 +170,7 @@ $.fn.position = function( options ) {
( rpercent.test( offsets.my[ 1 ] ) ? elem.outerHeight() / 100 : 1 )
],
collisionPosition,
using = options.using;
using;
if ( options.my[ 0 ] === "right" ) {
position.left -= elemWidth;
@@ -221,9 +221,9 @@ $.fn.position = function( options ) {
elem.bgiframe();
}
if ( using ) {
// we have to proxy, as jQuery.offset.setOffset throws away props other than left/top
options.using = function( props ) {
if ( options.using ) {
// adds feedback as second argument to using callback, if present
using = function( props ) {
var left = targetOffset.left - props.left,
right = (targetOffset.left + targetWidth) - (props.left + elemWidth),
top = targetOffset.top - props.top,
@@ -237,11 +237,11 @@ $.fn.position = function( options ) {
} else {
feedback.important = "vertical";
}
using.call( this, props, feedback );
options.using.call( this, props, feedback );
};
}
elem.offset( $.extend( position, { using: options.using } ) );
elem.offset( $.extend( position, { using: using } ) );
});
};