zuk/jquery.inview — pull request #4 by RSO (2013-01-14 16:00:24+00:00)
Title: Save event handlers

I've changed the behaviour of the jquery.inview plugin so that handlers aren't directly bound to the scroll event (like proposed on http://ejohn.org/blog/learning-from-twitter/).

Quote:

>**It's a very, very, bad idea to attach handlers to the window scroll event.** Depending upon the browser the scroll event can fire a lot and putting code in the scroll callback will slow down any attempts to scroll the page (not a good idea). Any performance degradation in the scroll handler(s) as a result will only compound the performance of scrolling overall. Instead it's much better to use some form of a timer to check every X milliseconds OR to attach a scroll event and only run your code after a delay (or even after a given number of executions - and then a delay).