autoDestroy kills triggers for all Instances about jquery.lazy HOT 8 CLOSED

can you create a fiddle? you could fork the one in #30 as a starting point (update the script src, it points to the feature branch)

from jquery.lazy.

Well, he's right. The problem is, that destroy() (as called on autoDestroy) removes all events by the namespace .lazy. There is no check or handling for multiple instances. The problem would be solved, if you include two script files with different names on line #550. 😄 But that should not be the workaround.

Maybe I implement an "instance count" witch will be added to the namespace. Or make the name configurable. But then the user had to known that behavior. Or something else. Have to think about that ...

from jquery.lazy.

for debugging single page apps, it would be nice to be able to pass in something meaningful for the namespace, like cid in backbone.js

from jquery.lazy.

There is a thing for this. It's the instance name. But the problem is, that it is actually not configurabe from the "outside". I should have to make this within an update ...

from jquery.lazy.

I fixed this issue in srcset branch and will be added to the master within the next update. Feel free to give it a try. Thans for reporting.

from jquery.lazy.

I would also need a fix for this. Is the plan that the srcset branch will replace the current master branch? If so I might as well use that one, even though I don't use responsive images..

from jquery.lazy.

Hello @razola,

yes, the srcset branch will be merged to the master as the next version. But the branch is actually not completely tested. The auto destroy problem is fixed, but there were made some more changes in the branch, like new features. So is should be working, but no guarantee. I think the updated will be released within the next 1-2 weeks.

If you want to use the actual srcset branch, I would recommend to use this particular version of Lazy, because it is an early state, with the auto destroy fixed:
https://github.com/eisbehr-/jquery.lazy/blob/140bc715ac202d8d37d9e8559c7209d057c01ca6/jquery.lazy.min.js

from jquery.lazy.

@eisbehr- thanks for the quick reply!

I'll use the version you specified until the next release.

from jquery.lazy.