Bug Tracker

Ticket #10676 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

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:
Blocking: Blocked by:

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

comment:1 Changed 3 years ago by dmethvin

  • Owner set to dmethvin
  • Priority changed from undecided to low
  • Status changed from new to assigned
  • Component changed from unfiled to event
  • Milestone changed from None to 1.7.1

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 3 years ago by Dave Methvin

  • Status changed from assigned to closed
  • Resolution set to fixed

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.