-
Notifications
You must be signed in to change notification settings - Fork 57
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
Added firefox & IE support + fixed callback not firing. #9
base: master
Are you sure you want to change the base?
Conversation
gijsroge
commented
May 22, 2015
- This will add support for Firefox and Internet Explorer.
- Add a check if scroll reached bottom page to trigger callback
Note: removed the hardcoded fix for scrolling with the body element. You should include this polyfill to fix the issue -> https://github.com/mathiasbynens/document.scrollingElement |
Why was this never merged? Does it still work? |
@derek-schultz Probably because they are busy which is perfectly reasonable, in the mean time you can use my fork. https://github.com/gijsroge/animated-scrollto |
@gijsroge I used your repository as replacing NPM package of this one, but it still does not work on FF |
@nikoloza Did you add the scollingElement polyfill? See: https://github.com/gijsroge/animated-scrollto#browser-support |