Opened 13 years ago
Closed 12 years ago
#5996 closed enhancement (wontfix)
Add attribute map support for .clone()
Reported by: | Licensed to Gil | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | core | Version: | 1.4.1 |
Keywords: | needsreview | Cc: | |
Blocked by: | Blocking: |
Description
Most of the time when I use the .clone() function, I'm cloning an element with an ID, so I have to change the ID with .attr() straight away.
I'd love to see the new $("<div/>", {attrMap}) syntax applied to .clone() as well, so that I could do:
$("#id").clone({id: "newID"});
Patch with unit tests here:
http://github.com/gilmoreorless/jquery/commit/5ad3237c11edf1e69e226f4e9bbefe9a53daa845
Cheers,
Gil
Change History (2)
comment:1 Changed 12 years ago by
Keywords: | needsreview added; clone manipulation removed |
---|---|
Status: | new → open |
comment:2 Changed 12 years ago by
Milestone: | 1.4.2 |
---|---|
Resolution: | → wontfix |
Status: | open → closed |
Agreed, explicit .clone().attr() is likely fine in this case.
Note: See
TracTickets for help on using
tickets.
Feature creep...marking for review. To me the explicit .clone().attr() is better.