Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#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.

http://jsfiddle.net/3cB95/

(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 dmethvin

Component: unfiledevent
Milestone: None1.7.1
Owner: set to dmethvin
Priority: undecidedlow
Status: newassigned

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.

comment:2 Changed 12 years ago by Dave Methvin

Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.