search

brandonaaron / livequery

MIT License
477 stars 195 forks source link

Seems to be broken with the latest jQuery 1.4 #2

Closed eych closed 14 years ago

eych commented 14 years ago

No test case sorry. For me, in some cases it doesn't work immediately (like an element matching the livequery selector is inserted already in DOM, but until I click inside a field on the page, livequery doesn't notice that), in some cases it works several times on the same element.

merbjedi commented 14 years ago

I'm also seeing this issue

whoahbot commented 14 years ago

We've seen this issue as well, and found a fix. We've posted an example and a fix for it at http://livequery14.heroku.com/

We'd be happy to fork and submit a pull request if needed.

eych commented 14 years ago

So, the fix seems to be to wrap livequery around "html" method call as well then?

whoahbot commented 14 years ago

That's what worked for us, changing $.livequery.registerPlugin to watch "html" as well.

merbjedi commented 14 years ago

Thanks for the fixes guys. Trying it out now. I think a fork + pull request would be extremely useful to others.

brandonaaron commented 14 years ago

Thanks whoahbot for the patch! Just applied it.

jonraasch commented 14 years ago

Thanks whoahbot, this really saved the day for me.