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

loader.getFile() returning undefined sometimes #35

Open
JamesMcFall opened this issue May 12, 2014 · 3 comments
Open

loader.getFile() returning undefined sometimes #35

JamesMcFall opened this issue May 12, 2014 · 3 comments

Comments

@JamesMcFall
Copy link

Hi Jussi,
I'm having an issue with the preloader where it seems to fail to create a .dom property on some select files after a certain point.

This didn't show up in my initial testing of it as it appeared to work fine for 10~ items to preload (a mix of images, mp3s and videos) but when I scaled up to about 100 files, inside the finish callback, loader.getFiles() started returning undefined for anywhere between 4-10 files (it varies between page refreshes).

Again this is due to not having a .dom property so getFile() can't match the file (despite it existing in the files array).

The files failing to load are a few mp3 and png files in this case.

Have you ever come across this behaviour?

@jussi-kalliokoski
Copy link
Owner

Hi James!

I haven't noticed this sort of behaviour before. Do you have a test case page somewhere I could try to see what's going on? Also, is this on all browsers or some specific one, e.g. Chrome?

@Aypac
Copy link

Aypac commented Aug 23, 2015

I do get it all the time actually. Using FF 40.0 on Ubuntu.
Only added JPG. Removing the .dom works fine, returning the whole object. When inspecting it in Firebug it has definitely a .dom .

I really don't get it :(

@jussi-kalliokoski
Copy link
Owner

Really sorry for neglecting this library - I haven't had the time to maintain it recently. :/ I suggest taking a look at alternatives such as pxloader that are more actively maintained and well battle-tested. :)

EDIT: that said, I will continue to review/merge incoming PRs and would be happy to give ownership to anyone willing to keep the project alive.

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