#15139 closed bug (fixed)

Opened February 16, 2017 02:04PM UTC

Closed February 21, 2017 07:31PM UTC

Last modified February 21, 2017 07:31PM UTC

Spinner: Mousewheel events should be ignored when the input is not focused

Reported by: s0By Owned by: scottgonzalez
Priority: minor Milestone: 1.12.2
Component: ui.spinner Version: 1.12.1
Keywords: Cc:
Blocked by: Blocking:
Description

I know this is somewhat intentional feature and is really annoying when the page is being scrolled by a user and, when the mouse gets over the spinner input, the page stops scrolling and spinning event starts inside the input.

This must be controlled. I bet in majority cases you would want to use a mousewheel scroll only when the input is actually focused. To comply for everyone, this could be made as an option, something simillar to what was done 7 years ago here:

http://btburnett.com/spinnertest/demos/spinner/mousewheel-disabled.html

I don't like to tamper with the solution and to keep overwriting the code as the version goes higher, good options could be included by original developers.

Attachments (0)
Change History (4)

Changed February 16, 2017 03:00PM UTC by scottgonzalez comment:1

status: new → open
summary: Spinner: Mousewheel event → Spinner: Mousewheel events should be ignored when the input is not focused

Changed February 19, 2017 03:54PM UTC by scottgonzalez comment:2

PR: https://github.com/jquery/jquery-ui/pull/1794

Changed February 21, 2017 07:31PM UTC by scottgonzalez comment:3

owner: → scottgonzalez
resolution: → fixed
status: open → closed

In [changeset:"a3b9129be19afabb3fa6b2fb913b85aab43f4652" a3b9129]:

#!CommitTicketReference repository="" revision="a3b9129be19afabb3fa6b2fb913b85aab43f4652"
Spinner: Ignore  events when not focused

Fixes #15139
Closes gh-1794

Changed February 21, 2017 07:31PM UTC by rjollos comment:4

milestone: none → 1.12.2