Opened 15 years ago
Closed 15 years ago
#3162 closed bug (invalid)
Typo in clone function causes error in IE
Reported by: | hjess | Owned by: | flesler |
---|---|---|---|
Priority: | major | Milestone: | 1.3 |
Component: | core | Version: | 1.2.6 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
After "ret" is assigned as the result of "this.map", events are processed according to "this", rather than to "ret".
Attachments (1)
Change History (5)
Changed 15 years ago by
Attachment: | jquery.diff added |
---|
comment:1 Changed 15 years ago by
need: | Review → Test Case |
---|
Eh.. I'm not sure that's wrong. I really can't follow that piece of code completely.
Can you provide a test case that shows the situation where this causes an error ? Just a simple html file with the needed js.
Thanks
comment:2 Changed 15 years ago by
After "ret" is assigned as the result of "this.map", events are processed according to "this", rather than to "ret".
Yes, because events are copied from this to the cloned elements. This is why this and it's childs must be visited.
comment:3 Changed 15 years ago by
Owner: | set to flesler |
---|---|
Status: | new → assigned |
Will check this out more closely when I get some time.
comment:4 Changed 15 years ago by
Resolution: | → invalid |
---|---|
Status: | assigned → closed |
It looks correct as-is; "this" is the original elements and clone is the cloned elements.
diff file