Modify ↓
Ticket #3344 (closed bug: duplicate)
wrapAll should probably call clone(true)
| Reported by: | waterson | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.3 |
| Component: | core | Version: | 1.2.6 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
Right now, wrap calls "clone()" instead of "clone(true)". This has the effect of causing any event handlers on the wrapping node to be lost; for example,
$("#someElement").wrap($("<div>").click(function() { alert("hi"); }));
ends up being equivalent to:
$("#someElement").wrap($("<div>"));
This is not hard to work around, but it would be nice to fix!
Attachments
-
3344.patch
(371 bytes) -
added by waterson 5 years ago.
- Proposed changes
Change History
Changed 5 years ago by waterson
-
attachment
3344.patch
added
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.
Proposed changes