Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added "eventData" argument support to Mousewheel event handler #10

Closed
wants to merge 1 commit into from

Conversation

nielmclaren
Copy link

Hi Brandon,

Thanks for providing the code for your jQuery mousewheel plugin.

I needed to pass additional data to the handler so I used code from jQuery's bind handler to implement the eventData argument.

Now you can do this, as with the other mouse events:

$.mousewheel({someVar: 42, prop: 'value'}, function (event, delta) {
alert(event.data.someVar);
});

Cheers,
Daniel

@shaneriley
Copy link

I'm going to assume this pull request was not accepted because of the lack of test additions, but even still why has this been sitting out as an open issue with a viable solution for nearly a year? I was just about to do the same as @DanielGM and add this in myself with test coverage, but if it's not going to be merged in I'll just leave the changes locally.

@zzzzBov
Copy link

zzzzBov commented Apr 27, 2012

Whoops, I made the same patch, and somehow didn't see this one first. Apologies for that.

@nielmclaren
Copy link
Author

No worries, thanks for the plugin!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants