From 4d2ac1abca64ec19407f4a8217388223aea9cca4 Mon Sep 17 00:00:00 2001 From: Scott González Date: Sat, 6 Aug 2011 13:27:42 -0400 Subject: Spinner: Don't stop propgation of mousedown. Added TODOs. --- ui/jquery.ui.spinner.js | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/ui/jquery.ui.spinner.js b/ui/jquery.ui.spinner.js index 1b55d8374..a8f3a4fe9 100644 --- a/ui/jquery.ui.spinner.js +++ b/ui/jquery.ui.spinner.js @@ -103,14 +103,15 @@ $.widget( "ui.spinner", { this._bind( this.buttons, { mousedown: function( event ) { if ( this._start( event ) === false ) { - // TODO: do we really want to stop propagation? - return false; + return; } + this._repeat( null, $( event.currentTarget ).hasClass( "ui-spinner-up" ) ? 1 : -1, event ); }, mouseup: function( event ) { if ( this.spinning ) { this._stop( event ); + // TODO: don't trigger change until the field is blurred this._change( event ); } }, @@ -125,7 +126,9 @@ $.widget( "ui.spinner", { } this._repeat( null, $( event.currentTarget ).hasClass( "ui-spinner-up" ) ? 1 : -1, event ); }, - // TODO: we shouldn't trigger any events until mouseup + // TODO: do we really want to consider this a stop? + // shouldn't we just stop the repeater and wait until mouseup before + // we trigger the stop event? mouseleave: function() { if ( this.spinning ) { this._stop( event ); @@ -183,6 +186,7 @@ $.widget( "ui.spinner", { this.timeout = setTimeout(function() { if ( this.spinning ) { this._stop( event ); + // TODO: don't trigger change until the field is blurred this._change( event ); } }, 100 ); -- cgit v1.2.3