Side navigation
#10676 closed bug (fixed)
Opened November 04, 2011 10:26PM UTC
Closed November 07, 2011 04:15PM UTC
Last modified March 08, 2012 08:01PM UTC
wheelDelta not added to mousewheel event anymore
Reported by: | anonymous | Owned by: | dmethvin |
---|---|---|---|
Priority: | low | Milestone: | 1.7.1 |
Component: | event | Version: | git |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
It appears that inside a mousewheel
event handler, wheelDelta
is not added as a property to the jQuery event object anymore, because mousewheel
is not in the list of events that have mouseHooks
added. Only some of the following events do, namely the ones that pass rmouseEvent
of the following:
"blur focus focusin focusout load resize scroll unload click dblclick " + "mousedown mouseup mousemove mouseover mouseout mouseenter mouseleave " + "change select submit keydown keypress keyup error contextmenu"
This renders the wheelDelta
property in mouseHooks
useless, since no other event passes this property. Only mousewheel
provides this data, but it is not used by jQuery because it's not in the above list.
It would perhaps be a better idea to either drop wheelDelta
from mouseHooks
or support mousewheel
as an event like the other mouse events above.
(Of course, e.originalEvent.wheelDelta
works; my point is that wheelDelta
is never used although it is in mouseHooks
.)
Attachments (0)
Change History (2)
Changed November 05, 2011 01:01AM UTC by comment:1
component: | unfiled → event |
---|---|
milestone: | None → 1.7.1 |
owner: | → dmethvin |
priority: | undecided → low |
status: | new → assigned |
I agree that it is not useful as is. Since it's not cross-browser it probably doesn't make sense to keep it on the list. Even if we added the mousewheel event it to mousehooks it wouldn't be cross-browser. Brandon Aaron's jquery-mousewheel.js plugin has been updated to work in 1.7 so I think we can remove this event from the list to copy.