Responsive faq accordion card [using HTML, CSS, JS]

Community feedback

• 

  finng1337•100

  @finng1337Posted about 2 years ago

  Nice work Moeka! I looked up the code and saw one thing in the javascript to improve: What if instead of looping through the all questions and removing the active state, just saving index of the opened question? That could be little faster solution. Keep going!

  Marked as helpful

  0

  ---

  

  rizkyizh•230

  @moe-kaPosted about 2 years ago

  Thank you for the advice

  is it true like this?

  const answers = document.querySelectorAll('.faq');
  var saveIndex = 0;
  
  answers.forEach(function (faq, index) {
      faq.addEventListener('click', function () {
          answers[saveIndex].classList.remove('active');
          answers[index].classList.toggle('active');
          saveIndex = index;
      })     
  })
  

  but the toggle() doesn't work

  0