Skip to content

Conversation

@gijsroge
Copy link

  • This will add support for Firefox and Internet Explorer.
  • Add a check if scroll reached bottom page to trigger callback

@gijsroge gijsroge closed this May 23, 2015
@gijsroge gijsroge reopened this May 23, 2015
@gijsroge gijsroge changed the title detect if browser supports document.body or document.documentElement detect if browser supports document.body or document.documentElement & fixed callback not firing in some cases May 23, 2015
@gijsroge gijsroge changed the title detect if browser supports document.body or document.documentElement & fixed callback not firing in some cases Added firefox & IE support + fixed callback not firing. May 23, 2015
@gijsroge
Copy link
Author

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

@derek-schultz
Copy link

Why was this never merged? Does it still work?

@gijsroge
Copy link
Author

gijsroge commented Apr 4, 2017

@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

@nikoloza
Copy link

@gijsroge I used your repository as replacing NPM package of this one, but it still does not work on FF

@gijsroge
Copy link
Author

@nikoloza Did you add the scollingElement polyfill? See: https://github.com/gijsroge/animated-scrollto#browser-support

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants