search

luis-almeida / unveil

A very lightweight jQuery plugin to lazy load images
4.16k stars 676 forks source link

Added debouncing for scroll event #93

Open aonic opened 9 years ago

aonic commented 9 years ago

https://developers.google.com/web/fundamentals/performance/rendering/debounce-your-input-handlers?hl=en

aonic commented 9 years ago

This change has been used in production for >5 months

aonic commented 9 years ago

What do you think @luis-almeida also look at diff without whitespace (not a lot of change) https://github.com/luis-almeida/unveil/pull/93/files?w=1

tantalum commented 9 years ago

We are also having issues with unveil and IE 9. We are looking at a solution similar to yours. We were just about to submit a pull request when we saw your pull request which is very similar to what we have done. I hope this gets merged soon so we can keep on using the official unveil plugin.