From a3b9129be19afabb3fa6b2fb913b85aab43f4652 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Sun, 19 Feb 2017 10:52:27 -0500 Subject: [PATCH] Spinner: Ignore `mousewheel` events when not focused Fixes #15139 Closes gh-1794 --- tests/unit/spinner/core.js | 34 +++++++++++++++++++++++++--------- ui/widgets/spinner.js | 6 +++++- 2 files changed, 30 insertions(+), 10 deletions(-) diff --git a/tests/unit/spinner/core.js b/tests/unit/spinner/core.js index 6d817134a..526f2ecf8 100644 --- a/tests/unit/spinner/core.js +++ b/tests/unit/spinner/core.js @@ -162,23 +162,39 @@ QUnit.test( "mouse click on up button, increases value not greater than max", fu } ); QUnit.test( "mousewheel on input", function( assert ) { - assert.expect( 4 ); + var ready = assert.async(); + assert.expect( 5 ); var element = $( "#spin" ).val( 0 ).spinner( { step: 2 } ); - element.trigger( "mousewheel" ); - assert.equal( element.val(), 0, "mousewheel event without delta does not change value" ); + element.simulate( "focus" ); + setTimeout( step1 ); - element.trigger( "mousewheel", 1 ); - assert.equal( element.val(), 2 ); + function step1() { + element.trigger( "mousewheel" ); + assert.equal( element.val(), 0, "mousewheel event without delta does not change value" ); - element.trigger( "mousewheel", -0.2 ); - assert.equal( element.val(), 0 ); + element.trigger( "mousewheel", 1 ); + assert.equal( element.val(), 2, "delta 1" ); + + element.trigger( "mousewheel", -0.2 ); + assert.equal( element.val(), 0, "delta -0.2" ); + + element.trigger( "mousewheel", -15 ); + assert.equal( element.val(), -2, "delta -15" ); - element.trigger( "mousewheel", -15 ); - assert.equal( element.val(), -2 ); + element.simulate( "blur" ); + setTimeout( step2 ); + } + + function step2() { + element.trigger( "mousewheel", 1 ); + assert.equal( element.val(), -2, "mousewheel when not focused" ); + + ready(); + } } ); QUnit.test( "reading HTML5 attributes", function( assert ) { diff --git a/ui/widgets/spinner.js b/ui/widgets/spinner.js index c63f4526a..0bc1db230 100644 --- a/ui/widgets/spinner.js +++ b/ui/widgets/spinner.js @@ -140,9 +140,13 @@ $.widget( "ui.spinner", { } }, mousewheel: function( event, delta ) { - if ( !delta ) { + var activeElement = $.ui.safeActiveElement( this.document[ 0 ] ); + var isActive = this.element[ 0 ] === activeElement; + + if ( !isActive || !delta ) { return; } + if ( !this.spinning && !this._start( event ) ) { return false; } -- 2.39.5