Great and simple plugin :D

here is my case, i have one class at first, then i loaded new element, but the scrollspy not updated. My new element didnt counted as scrollspy element. Any idea how to get this working?

Thanks in advance

The following changes are made:

• onScroll is throttled only once (otherwise two throttles happen independently of each other)
• not only one, but every onScroll is delayed until the DOM is ready (and throttled to ensure that onScroll is not called several times when the DOM becomes ready)
• each $.scrollSpy(...) call causes one onScroll call (throttled and DOM-delayed) regardless of the isSpying state (otherwise the second call and later $.scrollSpy(...) calls do not immediately trigger scrollSpy:enter on elements that are already “above the fold”)

The latter is actually a bugfix.

The entire viewport of the browser is currently used as the field; elements that enter this area have "entered" to use the terminology of this library.

I'm in need of offsets here - I would like to say that elements only enter when they've been scrolled half way up the viewport. This is not possible with the current design of code, and would probably need a bit of refactoring.

Preferably this would be something you could define in the options when calling the scrollSpy() method.

thanks for this great plugin. I just tested it, but unfortunately it doesn't work on content which is within a div with style overflow-y: scroll; maybe because the viewport sit's on window? any idea how to get this working??

Is there a way to set an offset of the element before/after entering the screen?

I am having trouble with spying on elements which are shorter than the window height and the next element enters - then the next element takes precendence over the element that was already in screen and didn't exit

You have a boo-boo in the example file.

You need to change:

    <script src="../scroll-spy.js"></script>

to:

    <script src="../scrollspy.js"></script>

Not a big deal. Any good developer can detect and fix this on his own.

Hi - great lib nice that materialize have started to use it. So I found you as I wanted an event for when the item is in the middle of the viewport. It almost feels faster to do it myself but would prefer to levarage and already in use lib ..thanks

Adding an option to provide offsets to how the viewport frame is calculated. As an example, this is useful when using a navbar that hides a part of the viewport.

@thesmart I have added this package to my package.json file

"dependencies": {
  ...
  "jquery-scrollspy": "https://github.com/thesmart/jquery-scrollspy.git"
  ...
}

when I run npm install there is an error npm ERR! premature close and this error because the package doesn't have a package.json file name, would you please add it ?

Hi I am using the following html:

`

and using the scrollSpy plugin as follows: ` $('#section2').on('scrollSpy:enter', function() { alert("scrolled in"); });

$('#section2').on('scrollSpy:exit', function() {
              alert("scrolled out");
});

$('#section2').scrollSpy();

`

When the page loads, even though #section2 is not visible the alert is shown by the browser. How can I avoid this? Also, if increase the height to 101% this issue is not occuring.

Hello @thesmart , I am the member of cdnjs project. We want to host this library. Please help me confirm the version on github. I saw the version of scrollspy.js is 0.1.2 in the release of 0.1.3. Thanks!

https://github.com/cdnjs/cdnjs/issues/8520

The lines of code using attribute selectors to match the id do not work with the latest jQuery:

$('a[href=#' + visible[0].attr('id') + ']')

In my case, the error thrown is: Error: Syntax error, unrecognized expression: a[href=#objective]

Simply changing the code to match the following correctly using " fixes this:

$('a[href="#' + visible[0].attr('id') + '"]')

I am happy to raise a PR to fix this as I feel it is quite important to address since Materialise CSS uses your plugin.