From ba4fa2b5ca43827f0deab5c241d0165208e8a4c8 Mon Sep 17 00:00:00 2001 From: jzaefferer Date: Mon, 7 Feb 2011 18:50:19 +0100 Subject: Spinner: Fix unnecessary var --- ui/jquery.ui.spinner.js | 1 - 1 file changed, 1 deletion(-) (limited to 'ui') diff --git a/ui/jquery.ui.spinner.js b/ui/jquery.ui.spinner.js index df61a7494..6d4d10d51 100644 --- a/ui/jquery.ui.spinner.js +++ b/ui/jquery.ui.spinner.js @@ -312,7 +312,6 @@ $.widget('ui.spinner', { }, _format: function(num) { - var num = this.options.value; this.element.val( $.global && this.options.numberformat ? $.global.format(num, this.options.numberformat) : num ); }, -- cgit v1.2.3 From ce51edefb326aa1838c2c62406a2ee2c8efb648a Mon Sep 17 00:00:00 2001 From: jzaefferer Date: Mon, 7 Feb 2011 20:37:06 +0100 Subject: Spinner: Fix time spinner demo; workaround for _super issue. --- ui/jquery.ui.spinner.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'ui') diff --git a/ui/jquery.ui.spinner.js b/ui/jquery.ui.spinner.js index 6d4d10d51..0219abb8b 100644 --- a/ui/jquery.ui.spinner.js +++ b/ui/jquery.ui.spinner.js @@ -285,11 +285,16 @@ $.widget('ui.spinner', { this.buttons.button("enable"); } } - this._super( "_setOption", key, value ); + // TODO see below + //this._super( "_setOption", key, value ); + $.Widget.prototype._setOption.apply( this, arguments ); }, _setOptions: function( options ) { - this._super( "_setOptions", options ); + // TODO _super doesn't handle inheritance with more then one subclass + // spinner subclass will have spinner as base, calling spinner._setOptions infinitely + //this._super( "_setOptions", options ); + $.Widget.prototype._setOptions.apply( this, arguments ); if ( "value" in options ) { this._format( this.options.value ); } -- cgit v1.2.3