-
Notifications
You must be signed in to change notification settings - Fork 22
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
"Too much recursion" error FF4 #2
Comments
See the comments on api.jquery.com/load-event/ for solution/alternative. |
I think I got that fixed locally, but I got issues replicating the image caching bugs in IE at the moment, so I'll wait to push it to github before actually verifying everything's working. |
I just ran into the recursion problem on FF3.6.10 (mac) too. It's inconsistent - it doesn't happen on every page load, and when it does it's on random elements. Using the workaround in the comments on the jquery api page (see above) until you push the update. |
Yes, as of now, you should be using either bind-event-before-setting-source approach, or the workaround on the API site. This plugin is right now busted without a clear solution, I'm afraid. |
The problem with this function is that it doesn't just bind to images - it adds this loader to every single element on the page. Change event.special.load to event.special.imgpreload, then bind an imgpreload event to the specific images you'd like to preload. Worked like a charm for me, in all browsers. |
jcljules's advice seems to work in Firefox 3.6 and Chrome 7. Thanks mate! |
I have an experimental fork of this plugin at https://github.com/jefferyto/jquery.imgloaded/tree/pi . Can you give it a try? I'd like to get some feedback before submitting a pull request. Thanks! |
When the plugin is run in FF4 this error is reported and the rest of the scripts on the page don't run.
The text was updated successfully, but these errors were encountered: