ressio / lazy-load-xt

Lazy load XT is a jQuery plugin for images, videos and other media
http://ress.io/lazy-load-xt-jquery/
MIT License
1.36k stars 245 forks source link

Class lazy-hidden not being removed from HTML5 video #27

Open dbhynds opened 9 years ago

dbhynds commented 9 years ago

For some reason, Lazy Load XT isn't removing the class .lazy-hidden on an HTML video that I have. I believe I've got the markup correct according to what you designate in the usage section. Lazy Load XT correctly replaces 'data-src' with 'src', and if I remove the class .lazy-hidden using the inspector, the video appears and works as expected. It just doesn't remove the class. It also throws now errors.

Download a zip here to see the problem in action: https://gist.github.com/dbhynds/055c83e497d5f2d583bb

dbhynds commented 9 years ago

Oh... I should note that this problem only happens if the fade-in effect is included (jquery.lazyloadxt.fadein.min.css)

If I remove this css file, it works as expected.

TomS- commented 8 years ago

Just come across this issue myself. Wouldn't mind the fix for this being pushed.

roelmagdaleno commented 6 years ago

Thanks, the commit 2408f26 works for this issue, also I tried 49a5988 at bryanbergman fork but didn't work for me.