I think, there is a little bug in your plugin. If you use $(this) in
the event function, your are pointing (in firefox) to the window
instead to the element.
Here is my code (firebug is your fried :):
$('.leaf').mousewheel(function(event, delta){
if (delta 0) {
console.info(Up);
Thanks Mathias! I've gone ahead and fixed it in SVN but haven't
updated the example/test yet.
http://svn.brandonaaron.net/svn/jquery_plugins/mousewheel.js
--
Brandon Aaron
On 10/5/06, Mathias Bank [EMAIL PROTECTED] wrote:
I think, there is a little bug in your plugin. If you use $(this) in
the
- The addition of mousewheelup/mousewheeldown methods.
Are these really necessary? I think adding these might increase the
code considerably without much more benefit. The mousewheelup/down is
represented by the event.detail and/or event.wheelDelta. Although the
event.detail and event.wheelDelta
- The addition of mousewheelup/mousewheeldown methods.
Are these really necessary? I think adding these might increase the
code considerably without much more benefit. The mousewheelup/down is
represented by the event.detail and/or event.wheelDelta. Although the
event.detail and
-
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
Behalf Of Brandon Aaron
Sent: Wednesday, September 27, 2006 11:04 PM
To: jQuery Discussion.
Subject: [jQuery] New Plugin: mousewheel
I've finished my mousewheel plugin + docs + test/example.
The example/test page:
http://brandonaaron.net/jquery
Safari in its current released version is hopeless.
I take that back. I'll do some testing and see what I can figure out
to make this work in Safari.
Also, sorry for not putting the [jQuery] tag on this thread and
continuing to reply to my own thread. :)
Brandon
Quote Andy Seems to work fine for me in IE6/PC. But I'm not sure what the difference
between one and two is.Test1...the box will scroll out of view (off the top of the page)whereas Test2 will remain staticthe page does not scroll
___
jQuery mailing
--//-
-Original Message-From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED]On Behalf Of Giuliano
MarcangeloSent: Thursday, September 28, 2006 8:52 AMTo:
jQuery Discussion.Subject: Re: [jQuery] New Plugin:
mousewheel
Quote Andy "Seems to work fine for me in
IE6/PC. Bu
Now that is works in the major browsers I'll think about adding the
mousewheelup and mousewheeldown but I'm still thinking it will just
add unecessary complication to the code.
I don't think it'll complicate it at all - it's like saying that
.mouseup() and .mousedown() are complicated and
Okay ... I've updated the plugin once again.
I overrode the mousewheel method to take an optional two functions
instead of one. The first function is the up handler and the second
function is the down handler. The preventDefault is still the last
param. I also added mousewheelup and
I overrode the mousewheel method...
I don't think I used the correct terminology there. That should be 'I
overloaded' instead of 'I overrode' ... at least I think so.
--
Brandon Aaron
On 9/28/06, Brandon Aaron [EMAIL PROTECTED] wrote:
Okay ... I've updated the plugin once again.
I overrode
I've finished my mousewheel plugin + docs + test/example.
The example/test page: http://brandonaaron.net/jquery/mousewheel/mousewheel.html
The code: http://svn.brandonaaron.net/svn/jquery_plugins/mousewheel.js
The blog entry:
http://brandonaaron.net/articles/2006/09/28/jquery-plugin-mousewheel
Great plugin. Couple recommendation:
- The addition of mousewheelup/mousewheeldown methods.
- The standardization of an event.detail and/or event.wheelDelta (IMO,
I think wheelDelta makes more sense). This way you can access the same
property in all browsers.
Do you still need testing for Opera
13 matches
Mail list logo
