Opened 14 years ago
Closed 14 years ago
#3820 closed bug (fixed)
.live() delegation fails in second handler if first handler modify the DOM
Reported by: | iraebrasil | Owned by: | brandon |
---|---|---|---|
Priority: | major | Milestone: | 1.3 |
Component: | event | Version: | 1.2.6 |
Keywords: | live, delegation, target | Cc: | |
Blocked by: | Blocking: |
Description
From the jQuery Development list: I'm using 1.3b2 (6070) and found the following behavior.
- using 2 .live() handlers for same link.
- 1st handler always fire and modify the link with .html(string)
- 2nd handler fire once, them stops firing.
I've made a test case: http://irae.pro.br/jquerytests/live.html.html
The weird thing is that after the manipulation, if you stop modifying the DOM (use toggle link in test case), both handlers fire again.
I dug into jQuery source but was unable to solve the problem myself to present a patch. But I found out that if I use .wrapInner() instead of .html() everything works fine. To me it seems that .closest() used in liveHandler can't match the element for delegation during domManip for some reason, but this was as far as I could get.
From: John Resig
Here is what I think is happening:
First click: Since there's only a text node inside of the link both callbacks work as expected - the target of the event is the anchor element. The one handler ends up modifying the contents with .html().
Second click: The first handler is run - but the target is the span, this time. You end up deleting the span while the event is still happening - stranding all the other events (they were targeted on an element that no longer exists).
For example, if you used .text("Click me") instead of the html business then this would've worked.
I don't really see a clear way to make this "just work". The target of the event was nuked while the event was still happening - we're kind of up the creek without a paddle on this one.
I'm open to thoughts, but I don't see a clear way to resolve this - at least not before the final 1.3 release.
Could you open a bug on this, at least? It'll be good to remember this issue.
Attachments (3)
Change History (4)
Changed 14 years ago by
Attachment: | live.html.html added |
---|
Changed 14 years ago by
Attachment: | 3820.patch added |
---|
patch to make a queue before firing every liveHandler solves #3820
Changed 14 years ago by
Attachment: | 3820.unit.patch added |
---|
Unit test proposal for this issue that the patch passes and without the path, don't
comment:1 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Version: | → 1.2.6 |
Fixed in SVN rev [6079] - thanks for the patches! Had to make a couple tweaks but they were good, thanks.
Test case for loosing the target for delegation but not a test unit