faq-accordion-card HTML, CSS, JS, flexbox

Community feedback

• 

  Eileen dangelo•1,600

  @EileenpkPosted over 2 years ago

  Excellent work, your code is very clean! Could you have used a forEach instead of running a loop on dropDown?

  dropDown.forEach((el) => { el.addEventListener('click', () => { if (el.classList.contains('active')) { dropDown.forEach((drop, index) => { el.classList.remove('active') }) } else { dropDown.forEach((drop, index) => { dropDown[index].classList.remove('active') }) el.classList.add('active') } }) })

  Marked as helpful

  0

  ---

  

  coc0dev•190

  @coc0devPosted over 2 years ago

  @Eileenpk Hi Eileen, I agree that is a little cleaner instead of switching loop styles. Thanks for pointing that out!

  0