sunnysideup / silverstripe-cms-niceties

dist-backup/js/app_MenuTabClick.js

window.addEventListener('load', function() {
    setTimeout(() => {
        if(window.location.hash) {
            const els = document.querySelectorAll('[aria-controls="'+window.location.hash.substring(1)+'"]');
            if(els.length) {
                el = els[0];

The variable el seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.el.

                if ("#"+el.getAttribute('aria-controls') === window.location.hash) {
                    el.firstChild.click();
                    el.firstChild.style.fontWeight = 'bold';
                    el.firstChild.style.color = '#0071c4';
                };
            }
        }
    }, 500);
});