Bug Tracker

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:


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:




Change History (2)

comment:1 Changed 12 years ago by dmethvin

Keywords: needsreview added; clone manipulation removed
Status: newopen

Feature creep...marking for review. To me the explicit .clone().attr() is better.

comment:2 Changed 12 years ago by john

Milestone: 1.4.2
Resolution: wontfix
Status: openclosed

Agreed, explicit .clone().attr() is likely fine in this case.

Note: See TracTickets for help on using tickets.