Javascript Onclick Event automatically execute

Asked Apr 03 '20 at 17:55

Active Apr 03 '20 at 18:01

Viewed 61 times

I add some buttons dynamically with javascript depending on the value of a select box. This is the code for the button:

    var cancel = document.createElement("button");
    cancel.innerHTML = "Cancel";
    cancel.type = "button";
    cancel.onclick = location.reload();
    container.appendChild(cancel);

This button is added to a modal dialog and should reload the page when it is clicked. If the function with which the button is added to the modal dialog is executed, the onclick event is also executed although the button was not clicked. Can anybody help me please? Thanks

asked Apr 03 '20 at 17:55

Max Mustermann

5

`location.reload();` will execeute immediatly. Omit brackets `()` or wrap it in function `() => location.reload();`. – m1k1o Apr 03 '20 at 17:57
@M1K1O's comment is the solution. – Sajeeb Ahamed Apr 03 '20 at 17:59
@M1K1O: Thanks for your help! :) – Max Mustermann Apr 03 '20 at 18:03
2

The recommended method is `cancel.addEventListener("click",location.reload)` or `cancel.addEventListener("click",function() { location.reload(); })` - note where the brackets are used here and where not – mplungjan Apr 03 '20 at 18:03
`location.reload();` return nothing.... – Mister Jojo Apr 03 '20 at 18:04

Javascript Onclick Event automatically execute

0 Answers0