simple-scrollspy icon indicating copy to clipboard operation
simple-scrollspy copied to clipboard

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

Open remcokalf opened this issue 3 years ago • 5 comments

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.

remcokalf avatar Dec 09 '21 14:12 remcokalf

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.

jboiss avatar Apr 11 '22 02:04 jboiss

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

kimyvgy avatar Oct 03 '22 05:10 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.

kimyvgy avatar Oct 10 '22 09:10 kimyvgy

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.

remcokalf avatar Oct 10 '22 11:10 remcokalf