One tiny optimization: classList.toggle has a little known optional second boolean argument which overrides the class being (in)active. So
if (windowY < lastScroll) { header.classList.add('scroll-up'); } else { header.classList.remove('scroll-up'); }
becomes
header.classList.toggle('scroll-up', windowY < lastScroll);
Oh nice actually never used that one, time to look into that Thanks Alex 💖
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
One tiny optimization: classList.toggle has a little known optional second boolean argument which overrides the class being (in)active. So
becomes
Oh nice actually never used that one, time to look into that
Thanks Alex 💖