-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
time to time observer not attached to DOM element #14
Comments
We now adopting https://github.com/maciekgrzybek/svelte-inview |
This should be in the readme :) I spent quite some time scratching my head about weird behavior with this one. Fixed by using bare IntersectionObserver. |
Actually there are no dependencies in it, all listed it devDeps, that needed to make bundle (rollup, typescript ts). There was missed peerDeps in it, but there is a PR to fix that. |
Currently intersection observer not always initialized after hydration.
For example - https://github.com/MailCheck-co/mailcheck.site/blob/master/src/components/Partners.svelte
It's live on https://www.mailcheck.co/ - if you will start scrolling during load it will never trigger.
That PR #13 supposed to fix such a behavioure, but seems like problem does not fixed by moving subscription to onMount handler.
The text was updated successfully, but these errors were encountered: