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/

Reply via email to