Fixed #6979 - Several components are leaking variables into the global scope

This commit is contained in:
Gilmore Davidson 2011-02-13 18:11:40 +11:00 committed by Scott González
parent c37637ee99
commit 9626484dd5
4 changed files with 5 additions and 5 deletions

View File

@ -15,8 +15,8 @@
$.effects.pulsate = function(o) {
return this.queue(function() {
var elem = $(this),
mode = $.effects.setMode(elem, o.options.mode || 'show');
times = ((o.options.times || 5) * 2) - 1;
mode = $.effects.setMode(elem, o.options.mode || 'show'),
times = ((o.options.times || 5) * 2) - 1,
duration = o.duration ? o.duration / 2 : $.fx.speeds._default / 2,
isVisible = elem.is(':visible'),
animateTo = 0;

View File

@ -140,7 +140,7 @@ $.effects.size = function(o) {
hProps = hProps.concat(['marginLeft','marginRight']); // Add margins
props2 = props.concat(vProps).concat(hProps); // Concat
el.find("*[width]").each(function(){
child = $(this);
var child = $(this);
if (restore) $.effects.save(child, props2);
var c_original = {height: child.height(), width: child.width()}; // Save original
child.from = {height: c_original.height * factor.from.y, width: c_original.width * factor.from.x};

View File

@ -542,7 +542,7 @@ if ( $.uiBackCompat !== false ) {
if ( this.options.navigation ) {
var self = this,
headers = this.element.find( this.options.header ),
content = headers.next();
content = headers.next(),
current = headers.add( content )
.find( "a" )
.filter( this.options.navigationFilter )

View File

@ -592,7 +592,7 @@ $.widget( "ui.slider", $.ui.mouse, {
return this._valueMax();
}
var step = ( this.options.step > 0 ) ? this.options.step : 1,
valModStep = (val - this._valueMin()) % step;
valModStep = (val - this._valueMin()) % step,
alignValue = val - valModStep;
if ( Math.abs(valModStep) * 2 >= step ) {