Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.8.3 - new initial update() behavior crashes image loading #88

Closed
flopi opened this issue Jan 9, 2013 · 3 comments
Closed

1.8.3 - new initial update() behavior crashes image loading #88

flopi opened this issue Jan 9, 2013 · 3 comments

Comments

@flopi
Copy link

flopi commented Jan 9, 2013

After upgrading from 1.8.2 to 1.8.3 (to address the Safai/Webkit initial loading bug),
images (using a container in my case) did not load dynamically.

To get both working (Safari and the dynamically loading) I simply copied the old function back in:

Line #147:
/* Force initial check if images should appear. */
$(window).load(function() {
update();
});
$(document).ready(function() {
update();
});

just FYI the function I use to "dynamically" assign a lazyload (as I have around 100 containers with several pictures on one page, I cannot initialize all lazyloaders at once due to performance):

$("img.lazy").show().lazyload({
effect : "fadeIn",
placeholder: "images/loading.gif",
container: $('#scroller'+scroller)
});

while the sroller variable is the id of the container that I want to load dynamically after an event fires (click, scroll).

@tuupola
Copy link
Owner

tuupola commented Jan 9, 2013

URL of the page you have problem with?

@giver
Copy link

giver commented Jan 30, 2013

I have the same problem.

This code work on Lazyload 1.8.2 but not on 1.8.3.

$('#take-photo-button').before(output); // 'output' contain <img> that has class "lazy"
$('img.lazy').lazyload();

Note: I tested on jQueryMobile 1.2.0 with Android 4.0.4

@tuupola
Copy link
Owner

tuupola commented Aug 9, 2013

Should be fixed in 1.8.5. Could you test with the latest and reopen the issue if you still experience problems.

@tuupola tuupola closed this as completed Aug 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants