Feature request: remove active class on first item in nav, when scrolled back up

[remcokalf]

Currently, when scrolling down, active classes are added and removed. Then scrolling back up, the last item (first in the nav) keeps its active class, even if the correponding section is way past it.

This looks confusing, especially in a page where the content + nav are a bit lower, e.g. below a large hero section.
When you scroll back up, and the section is way low, the first item still keeps its active class.

[jboiss]

This is a very useful and lightweight scrollspy script, so thanks for your work. I think it would be perfect if not for this issue, which appears as a bug to the end user.

[kimyvgy]

@jboiss @remcokalf Excuse me! I'm sorry for late reply. Could you, please give me a capture image for this issue?

[kimyvgy]

This thread has been locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

[remcokalf]

Hi, was this issue fixed? Or just closed? I don't see any commits related to it? I don't understand why you asked for a screen capture. Was the description not clear? I can provide a capture of the issue next week.

[kimyvgy]

I am reopening this issue as the previous fix has led to a more critical bug (#59). We need to explore alternative solutions. The problem persists in version 2.5.1.

Screen.Recording.2024-09-18.at.23.30.56.mov