Continuously scrolling is not responsive enough

[Martinsos]

Hi, first of all, thank you for creating this great module!
I noticed a possible bug: inview does not trigger until user does not stop scrolling. So if user is continuously scrolling (for example using touchpad on laptop), inview does not trigger. Can you confirm that is happening? Is that expected behaviour and limitation of angular inview, or a bug?
Thank you!

[ryanmclaughlin]

I'm experiencing this as well.

[4storia]

Line 256 of the source has
return timer = setTimeout(f, t != null ? t : 100);
If you need the scroll event to fire more frequently, lower the 100 to something smaller, ie
return timer = setTimeout(f, t != null ? t : 40);

[thenikso]

would you think parametrizing that debounce timer would make sense? possibly setting it at 0 to remove the debounce entirely.

What would a good way of parametrizing it look like to you? Would a provider be too much? maybe a constant would suffice. opinions?

[arthur-white]

On second thought,
as soon as the scrolling begins /* Turn on polling every n miliseconds /
When the scrolling stops / Do a debounced stop polling call */

[Martinsos]

Is javascript executing while scrolling is happening?

[thenikso]

there is no polling, only debouncing of scroll events to 100ms. Javascript is always executing when scrolling (now also on iOS8).

[urecio]

Hi, I think this could be better if we can configure it.
I have created a pull request
#35

[thenikso]

I was planning to introduce another breaking change (I like those) by deprecating in-view-offset and add in-view-options (much like the new ng-model-options) which may work like:

<any in-view="$inview&&someFunction()" in-view-options="{ offsetTop:100, offsetBottom:0, debounce:100 }"></any>

The default debounce value may be 0 (meaning no debounce at all).

[thenikso]

With the PR #36 now published as version 1.5.0 this should be fixed. Tests are appreciated!