You're correct, but I've been developing against jQuery 0.x which doesn't have extend and for some reason the keydown/keyup events weren't firing...
I plan to release a 1.x compatible version.
-js
On 9/19/06, Jörn Zaefferer <[EMAIL PROTECTED]> wrote:
Jonathan Sharp schrieb:
> Feedback appreciated!
You could optimize the lines:
for (o in opts) { $.autoscroll.settings[o] = opts[o]; }
to
$.extend( $.autoscroll.settings, opts );
In addition, it would be nice if you would use jQuery's event handling, instead of just assigning plain events to document.onmousemove etc., like this:
$(document).mousemove($.autoscroll.setMouseEvent).keydown($.autoscroll.setKeyEvent).keyup(...);
-- Jörn
_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/
_______________________________________________ jQuery mailing list [email protected] http://jquery.com/discuss/
