Mike
Mike

Reputation: 31

Cloning elements [div] [script] in Javascript

I'm trying to clone a div in pure Javascript, however, cloneNode leads to duplicate ids (div_0). I would like to increment the id as div_1, div_2... and do the same to somevar = {'elem', 'div_1'}... Thanks

<html>
<head></head>
<body>
<div id="mydiv">
    <div id="div_0">
        <script type="text/javascript">
            <!--
            somevar = {'elem', 'div_0'};
            //-->
        </script>
        <p>HELLO</p>
    </div>
</div>

<a href="#" onclick="cloning()">CLONE</a>

<script type="text/javascript">
    function cloning() {
        var container = document.getElementById('mydiv');
        var clone = document.getElementById('div_0').cloneNode(true);
        container.appendChild (clone);
    }
</script>
</body>
</html>

Upvotes: 3

Views: 9311

Answers (3)

Kai
Kai

Reputation: 9288

I haven't had a chance to test this, but this might do the trick:

function cloning (e) {
    if (typeof e === 'string') {
        e = document.getElementById(e);
    }

    var clone = e.cloneNode(true),
        last_inc = cloning.last_inc || parseInt(e.id.match(/(\d+)$/)[0], 10);

    last_inc += 1;

    clone.id = 'div_' + last_inc;
    cloning.last_inc = last_inc;

    e.parentNode.appendChild(clone);
}

If you want to update a global variable somewhere, you can do return { "elem" : "div_" + last_inc }; and assign the result of the function. Or instead of a return, you could explicitly make the assignment. I'd recommend the former.

Upvotes: 0

Jishnu A P
Jishnu A P

Reputation: 14382

Try this..

`

function cloning() {
    var container = document.getElementById('mydiv');
    var clone = document.getElementById('div_0').cloneNode(true);
    clone.setAttribute('id','div_'+document.getElementById('mydiv').getElementsByTagName('div').length);
    container.appendChild (clone);
}

`

Upvotes: 4

meder omuraliev
meder omuraliev

Reputation: 186572

Set counter=0 and then increment after each cloning invocation with counter++ and use clone.id and set it to 'div_' + counter

Upvotes: 1

Related Questions