85

I tried

node.cloneNode(true); // deep copy

It doesn't seem to copy the event listeners that I added using node.addEventListener("click", someFunc);.

We use the Dojo library.

Brett DeWoody
  • 59,771
  • 29
  • 135
  • 184
Chakradar Raju
  • 2,691
  • 2
  • 26
  • 42

9 Answers9

101

cloneNode() does not copy event listeners. In fact, there's no way of getting hold of event listeners via the DOM once they've been attached, so your options are:

  • Add all the event listeners manually to your cloned node
  • Refactor your code to use event delegation so that all event handlers are attached to a node that contains both the original and the clone
  • Use a wrapper function around Node.addEventListener() to keep track of listeners added to each node. This is how jQuery's clone() method is able to copy a node with its event listeners, for example.
Tim Down
  • 318,141
  • 75
  • 454
  • 536
  • Do you perhaps have an example of code how to add event listeners to the cloned code? –  Jun 08 '18 at 08:32
  • @RoelofCoertze - you would have to know which events listeners were added in the first place and also have access in the code's place where you've done the cloning, to assign the callbacks again, since accessing the callbacks might be tricky depending where the cloning took place in the code, so an example will be useless since there are too many scenarios. – vsync Jun 26 '18 at 16:07
  • Currently faced with this issue. It seems that cloneNode behaves quite tricky when we are talking about event listeners copy: `It does not copy event listeners added using addEventListener() or those assigned to element properties (e.g., node.onclick = fn).` – Ihor Pavlyk Sep 12 '19 at 04:55
  • 2
    @jeromej it is not always a bad habit: there are use cases when separating JS from HTML is undesirable, i.e. `
    ` loaded by AJAX means race hazard (and accidental complexity) if you separate it. Separating is good for extending reasons, but some code can be sealed by design. Both content attributes and IDL attributes exist for a reason.
    – Jan Turoň Nov 06 '20 at 19:51
17

This does not answer the question exactly, but if the use case allows for moving the element rather than copying it, you can use removeChild together with appendChild which will preserve the event listeners. For example:

function relocateElementBySelector(elementSelector, destSelector) {
  let element = document.querySelector(elementSelector);
  let elementParent = element.parentElement;
  let destElement = document.querySelector(destSelector);
  elementParent.removeChild(element);
  destElement.appendChild(element);
}
Richard Greenwood
  • 876
  • 1
  • 10
  • 20
2

Event Delegation example.

After reading Tim Down's answer, I found delegated events are very easy to implement, solving a similar problem I had. I thought I would add a concrete example, although it's in JQuery not Dojo.

I am re-skining an application in Semantic UI, which requires a small piece of JS to make the message close buttons work. However the messages are cloned from an HTML template tag using document.importNode in a library. This meant even if I did attach the event handlers to the template in the new HTML, they are lost during the cloning.

I cannot do Tim's option 1, to simply re-attach them during cloning as the messaging library is front-end framework agnostic. (Interestingly my previous front-end was in Zurb Foundation which uses a "data-closable" attribute, the functionality of which does survive the cloning process).

The normal event handling suggested was like this:

$('.message .close').on('click', function() {
    $(this)
    .closest('.message')
    .transition('fade');
});

The problem being ".message" at app-load only matches the single template, not the actual messages which arrive later over web-sockets.

Making this delegated, meant attaching the event to the container into which the messages get cloned <div id="user-messages">

So it becomes:

$('#user-messages').on('click', '.message .close', function() {
    $(this)
    .closest('.message')
    .transition('fade');
});

This worked immediately, saving any complex work like the third option of wrapping the event subs.

The Dojo equivalent looks pretty similar in concept.

scipilot
  • 6,681
  • 1
  • 46
  • 65
2

Only inline attributes would work here which are heavily, heavily discouraged because of how misused they are. That said, you can have elements bind to the same event listener.

The proper way with Web Components (and shadow root) would look like and what we would want to replicate:

onButtonClick(event) {
  console.log('onButtonClick', { event, this: this });
}

/* Or constructor */
connectedCallback() {
  this.shadowRoot.getElementById('button')
    .addEventListener(this.onButtonClick);
}

It's efficient because you don't create function per element like you would with .addEventListener(() => this.onButtonClick). 1000 buttons would attach to the same function instead of creating a new function per button.

To convert that to inline would look like this:

<button onclick="this.getRootNode().host.onButtonClick.call(this, event)">

Is it ugly? Yes. But does it work? Also, yes. In this case there's no need for JS to have find the element and instruct the browser to create an event handler. The inline onclick does that for you. I will note that are creating a new function for each and every element, instead of them all sharing one.

ShortFuse
  • 5,970
  • 3
  • 36
  • 36
1

This is what @JeromeJ was describing in a comment. Create the initial element using this HTML code.

<DIV ONCLICK="doSomething(this)">touch me</DIV>

When you clone this element the result will have the same handler, and "this" will point to the cloned element.

It would be great if the ONCLICK handler could easily be added in JavaScript. This approach means that you have to write some of your code in HTML.

Trade-Ideas Philip
  • 1,067
  • 12
  • 21
  • This is also called content attribute, which is independent to [IDL attribute](https://developer.mozilla.org/en-US/docs/Glossary/IDL). – Jan Turoň Nov 06 '20 at 19:39
1

Cloning a node copies all of its attributes and their values, including intrinsic (inline) listeners. It does not copy event listeners added using addEventListener() or those assigned to element properties (e.g., node.onclick = someFunction). Additionally, for a element, the painted image is not copied.

source: MDN (https://developer.mozilla.org/en-US/docs/Web/API/Node/cloneNode).

Raki Lachraf
  • 99
  • 1
  • 8
1
const _originAddEventListener = HTMLElement.prototype.addEventListener;
const _originRemoveEventListener = HTMLElement.prototype.removeEventListener;
const _originCloneNode = HTMLElement.prototype.cloneNode;
const _eventListeners = [];

const getEventIndex = (target, targetArgs) => _eventListeners.findIndex(([elem, args]) => {
    if(elem !== target) {
        return false;
    }

    for (let i = 0; i < args.length; i++) {
        if(targetArgs[i] !== args[i]) {
            return false;
        }
    }

    return true;
});

const getEvents = (target) => _eventListeners.filter(([elem]) => {
    return elem === target;
});

const cloneEvents = (source, element, deep) => {
    for (const [_, args] of getEvents(source)) {
        _originAddEventListener.apply(element, args);
    }

    if(deep) {
        for(const i of source.childNodes.keys()) {
            const sourceNode = source.childNodes.item(i);
            if(sourceNode instanceof HTMLElement) {
                const targetNode = element.childNodes.item(i);
                cloneEvents(sourceNode, targetNode, deep);
            }
        }
    }
};

HTMLElement.prototype.addEventListener = function() {
    _eventListeners.push([this, arguments]);
    return _originAddEventListener.apply(this, arguments);
};

HTMLElement.prototype.removeEventListener = function() {

    const eventIndex = getEventIndex(this, arguments);

    if(eventIndex !== -1) {
        _eventListeners.splice(eventIndex, 1);
    }

    return _originRemoveEventListener.apply(this, arguments);
};

HTMLElement.prototype.cloneNode = function(deep) {
    const clonedNode = _originCloneNode.apply(this, arguments);
    if(clonedNode instanceof HTMLElement){
        cloneEvents(this, clonedNode, deep);
    }
    return clonedNode;
};
  • As it’s currently written, your answer is unclear. Please [edit] to add additional details that will help others understand how this addresses the question asked. You can find more information on how to write good answers [in the help center](/help/how-to-answer). – Community Apr 25 '23 at 18:16
0

Though it's impossible to clone a node altogether with its event listeners, it's possible to pass the event back to the origin element, to get similar effects.

Here's an implementation of this idea: https://gist.github.com/MicroCBer/a412a6c27a6dc0e49c091ce64e96ae6b

You can test it through this script:

// run in any Stackoverflow question pages with answers.

// This would create a perfect delegate dom of answers element, all the buttons should be interactable.
// sadly, the keyboard input is not working

const win = window.open('about:blank', undefined, 'popup');

for(const style of document.querySelectorAll('style, link')) 
    win.document.head.appendChild(style.cloneNode(true));

win.document.body.appendChild(createElementDelegate(document.querySelector("#answers"), {cloneParent:true, syncProps: true}))
<script src="https://gist.githubusercontent.com/MicroCBer/a412a6c27a6dc0e49c091ce64e96ae6b/raw/9553ffa4af0d79980212d7fdf9c884cdbf3f3d16/cloneEx.js"></script>
MicroBlock
  • 36
  • 1
  • 3
-1

I know I'm late to the party but this a solution that worked for me:

const originalButtons = original.querySelectorAll<HTMLElement>('button');
const cloneButtons = clone.querySelectorAll<HTMLElement>('button');
originalButtons.forEach((originalButton: HTMLElement, index: number) => {
  cloneButtons[index].after(originalButton);
  cloneButtons[index].remove();
});
Obtuse
  • 1
  • 2
  • 1
    how does this answer the question? what is `original` and `clone` objects? and more important, the events from the `original` are not copied into the `clone` element in any way – Soldeplata Saketos Jan 20 '23 at 07:08