accordion.js

export default accordion;

function accordion(arrayOfAccordions = []) {
    for(let i = 0; i<arrayOfAccordions.length; i++) {
        arrayOfAccordions[i].addEventListener('click', (el) => {
            if(el.target.classList.contains('js-accordion-toggler')) {
                el.target.parentNode.classList.toggle('accordion__item--expanded');
            }
            return false;
        });
    }
}

/*
    Given following markup
    <ul class="accordion">
        <li className="accordion__item">
            <a class="js-accordion-toggler">
                // TOGGLER CONTENT
            </a>
            <div class="accordion__content">
                // CONTENT HERE
            </div>
        </li>
    </ul>

    <style>
        .accordion__content {
            display: none;
        }
        .accordion__item--expanded .accordion__content {
            display: block;
        }
    </style>
*/

el.target.classList.value.indexOf('js-accordion-toggler')) {

går också att skriva som

el.target.classList.contains('js-accordion-toggler')

Varför anropar du toggleAccordionItem när det bara är en rad kod som du kan köra ist för funktionsanropet? Toggler som du skickar med används inte.

Ska denna byggas ut?

Updaterade efter dina kommentarer