#10676 closed bug (fixed)
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
.)
Change History (2)
comment:1 Changed 12 years ago by
Component: | unfiled → event |
---|---|
Milestone: | None → 1.7.1 |
Owner: | set to dmethvin |
Priority: | undecided → low |
Status: | new → assigned |
comment:2 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fix #10676. Remove wheelDelta
from the event props list.
A cross-browser solution is best handled in a plugin such as jquery-mousewheel.js .
Changeset: 13aa84580a837b811dbe2f5d1bc34cf9ca9a6d8d
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.