Passive scroll listener about stickybits HOT 5 CLOSED

@aFarkas You are right, I assumed scroll was preventable, apparently it's only touch and wheel events. However if you don't mind, that bomb is more than welcome.

Even then, it's not a misunderstanding on my part. It was due to partially missing knowledge.

from stickybits.

Thx, @anilanar. I just answered this stackoverflow question which I found after looking how many people are using this technique.

Checkout the rAF part. Because you are using the non useful pattern there too. (It's also related to this one).

By the way you need to update getBoundingClientRect from time to time otherwise it can't react to other layout change like scripted widgets that influences height of the document or window resize...

from stickybits.

@anilanar, I did not know about the passive property but the pr looks 👌! I will look shortly 🙏

from stickybits.

Using the passive event option for the scroll event doesn't make much sense. If used in conjunction with scroll event it only shows some basic misunderstanding of the devs writing the code.

happy to drop some knowledge bombs about scroll, passive and stuff if you want.

from stickybits.

@aFarkas thanks for your insights. 🤓

from stickybits.