So if I want something to happen when I click a button I can either do:

<!-- EXAMPLE A-->
<input type="button" onclick="alert('hello world');">

or I can do

<!-- EXAMPLE B-->
<input type="button" id="clickme">
<script>
   $('#clickme').on("click",function(){alert('hello world');});
</script>

Or of course any variations (on change, on hover) and shortcuts (.click() .change()) are possible...

Besides the fact that A is shorter, what are the differences? Which is better and why?

I noticed, that when I use .html() to dynamically add an element to the site (like a button) B doesn't work for this newly created button, and I need to use A...

Any insights would be helpful!

So if I want something to happen when I click a button I can either do:

<!-- EXAMPLE A-->
<input type="button" onclick="alert('hello world');">

or I can do

<!-- EXAMPLE B-->
<input type="button" id="clickme">
<script>
   $('#clickme').on("click",function(){alert('hello world');});
</script>

Or of course any variations (on change, on hover) and shortcuts (.click() .change()) are possible...

Besides the fact that A is shorter, what are the differences? Which is better and why?

I noticed, that when I use .html() to dynamically add an element to the site (like a button) B doesn't work for this newly created button, and I need to use A...

Any insights would be helpful!

• javascript
• jquery
• html
• event-handling

• go here and you can find it it's a same question – Chow Commented May 13, 2017 at 6:56

4 Answers 4

<input type="button" onclick="alert('hello world');">

This is the way how inline events are handled. The main reason this is a bad idea is to clearly define the separation of concerns.

HTML - Structure of your page

JS   - Your page functionality

This will cause less maintenance issues in the long run and when the system scales in size.

What happens if you have 100 buttons on your page and you want to remove the click event or change it for all of them. It would definitely be a nightmare. Also you can only define a single event if you bind it inline.

By moving out to a separate file you have a lot of flexibility and you can just make a small change that will affect all the elements on the page.

So the 2nd approach is always better. and the way to go.

By defining the events like below

$('#clickme').on("click",function(){alert('hello world');});

you HTML looks clean sans of any functionality and removes the tight coupling.

In the cases you have a dynamically added, it is true inline events always work but there is a concept called Event Delegation . You attach the event to a parent container that is always present on the page and bind the event to it. When the event occurs at a later time on the element , the event bubbles to the parent which handles the event for you.

For such cases you bind the events using .on passing in a selector

$(document).on('click', '#clickme', function() {

Keep in mind that binding multiple events to a document is a bad idea. You can always use the closestStaticAncestor to bind the events.

The first approach only lets you register one click listener whereas the second approach lets you register as many as you want.

If you want clicks on dynamically added elements to be heard as well, you should use .on(). Here's some code that demonstrates this (jsfiddle):

HTML

<div id="mainDiv">
<span class="span1">hello</span>
</div>

JS

$(document).on("click", ".span1", function () {
   $( "#mainDiv" ).append("<span class=\"span1\">hello</span>");
});

$(".span1").click( function () {
   console.log( "first listener" );
}); 

$(".span1").click( function () {
   console.log( "second listener" );
}); 

Notice that first listener and second listener is only printed when clicking on the first hello, whereas a new span gets added when clicking on any of the spans.

Guessing this is your real question:

I noticed, that when I use .html() to dynamically add an element to the site (like a button) B doesn't work for this newly created button, and I need to use A...

Simply use the selector in the .on() function, AND USE A CLASS instead of DUPLICATE IDs for multiple elements:

$('document').on("click", ".myButtons", function(){
    alert('hello world');
});

Button(s) - change to class (if you use IDs, only the FIRST will be selected):

<input type="button" class="myButtons" />
<input type="button" class="myButtons" />

This is how you should use .on() to attach event handlers to new elements:

https://stackoverflow.com/a/14354091/584192

Difference in works, if you use click() you can add several fn, but if you use attribute only one function will be executed - the last one

HTML-

<span id="JQueryClick">Click #JQuery</span> </br>
<span id="JQueryAttrClick">Click #Attr</span> </br>

Js-

$('#JQueryClick').on("click", function(){alert('1')});
$('#JQueryClick').on("click", function(){alert('2')});

$('#JQueryAttrClick').attr('onClick'," alert('1')" );//this doesn't work    
$('#JQueryAttrClick').attr('onClick'," alert('2')" );

If we are talking about performance, in any case directly using is always faster, but using of attribute you will be able to assign only one function.

Try This

Reference