Want to add "addEventListener" on multiple elements with same class

Question

I'd like to perform the logic in the "onClick" through the event listener in jS but it only seems to run once? I have the class in all four but I can't figure out why it seems to only work for the first?

HTML:

<button id='btn-1' type="button" name="first" class="breakdown main-text" onclick="enableButton('btn-2');disableButton('btn-1');show('btn-1')"> Breakdown Start </button>
<button id='btn-2' type="button" name="second" class="breakdown main-text" onclick="enableButton('btn-3');disableButton('btn-2');show('btn-2')" disabled> Repair Start </button>
<button id='btn-3' type="button" name="third" class="breakdown main-text" onclick="enableButton('btn-4');disableButton('btn-3');show('btn-3')" disabled> Repair End </button>
<button id='btn-4' type="button" name="fourth" class="breakdown main-text" onclick="show('btn-4')" disabled> Breakdown Ended </button>

JS:

let button1 = document.querySelector('#btn-1')
let button2 = document.querySelector('#btn-2');
let button3 = document.querySelector('#btn-3');
let button4 = document.querySelector('#btn-4');


const breakdownButton = document.querySelector('.breakdown');
breakdownButton.addEventListener('click', function() {
console.log(this.innerHTML);
});

The other buttons are disabled. Event listener for them won't be triggered apart from the first button — kboul, Jul 28 '18 at 17:04
How else would I dynamically disable and re enable the buttons? — Nakul Malhotra, Jul 28 '18 at 17:26

score 6 · Answer 1 · answered Jul 28 '18 at 17:02

You need to use querySelectorAll which will return a collection.Now use spread operator (three dots) to convert it to array and use forEach .Inside forEach callback add the event listener to it

[...document.querySelectorAll('.breakdown')].forEach(function(item) {
  item.addEventListener('click', function() {
    console.log(item.innerHTML);
  });
   });

<button id='btn-1' type="button" name="first" class="breakdown main-text"> Breakdown Start </button>
<button id='btn-2' type="button" name="second" class="breakdown main-text" disabled> Repair Start </button>
<button id='btn-3' type="button" name="third" class="breakdown main-text" disabled> Repair End </button>
<button id='btn-4' type="button" name="fourth" class="breakdown main-text" disabled> Breakdown Ended </button>

In your snippet you have also attached inline event handler,that may not be necessary.

If the objective is to enable the next button then a function to enable it can be called from the callback function of the event handler

Is the spread into an array really necessary? `forEach` can be called on the `querySelectorAll` result in Chrome console it would seem. — benmccallum, Jun 13 '19 at 09:08
It cannot however be called directly on the result of `getElementsByClassName` unfortunately... — benmccallum, Jun 13 '19 at 09:20

score 2 · Answer 2 · answered Jul 28 '18 at 17:01

2

Look at the documentation for querySelector:

querySelector() returns the first Element within the document that matches the specified selector, or group of selectors.

If you want to match more than one element, you'll need to use querySelectorAll and, because it doesn't return a single element loop over the result.

Alternatively, you could use event delegation.

answered Jul 28 '18 at 17:01

Quentin

800,325
104
1,079
1,205

thank you! Is one more valuable than the other in certain cases? – Nakul Malhotra Jul 28 '18 at 17:09
@NakulMalhotra — Benefits of event delegation are explained in detail on the question I linked to. – Quentin Jul 28 '18 at 17:11

htoniv · Answer 3 · 2019-06-24T07:34:56.783

0

Need to use querySelectorAll instead of querySelector. And iterate over the list like this.

const breakdownButton = document.querySelectorAll('.breakdown');

// It add event listeners for the first button element. 
// you can use forloop or using map function to iterate for the list elements and here i used breakdownButton[0] as an example.

breakdownButton[0].addEventListener('click', function() {
  console.log(this.innerHTML);
});

Use iterate functions like forEach or map. I used forEach

const breakdownButton = document.querySelectorAll('.breakdown');
breakdownButton.forEach(function(btn) {
  btn.addEventListener('click', function() {
    console.log();
  });
});

edited Jun 24 '19 at 07:34

answered Jul 28 '18 at 17:02

htoniv

1,550
12
34

`[0]` doesn't iterate over the list. It just takes the first element in it. – Quentin Jul 28 '18 at 17:03
no like this we need to add event listener for the button elements i given the example. – htoniv Jul 28 '18 at 17:06

Want to add "addEventListener" on multiple elements with same class

3 Answers3

Linked

Related