Skip to content
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

Update js/jquery.smoothDivScroll-1.3.js #48

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

css117
Copy link

@css117 css117 commented Oct 3, 2012

Adding beforeEndDistance option to the object, to trigger scrollerRightLimitSoon callback.

Adding beforeEndDistance option to the object, to trigger scrollerRightLimitSoon callback.
@tkahn
Copy link
Owner

tkahn commented Oct 4, 2012

This callback is triggered not only when the scroller reaches the point but on every following point. I think this could be a problem if you use it to load content through AJAX(?)

edit

I think I have a solution for this.

@css117
Copy link
Author

css117 commented Oct 4, 2012

In my context, the triggered action is handled by Drupal Ajax : it avoids multiple submission.
So it's not a problem for me, but it could be in some less robust script.

@tkahn
Copy link
Owner

tkahn commented Oct 5, 2012

I see! If I can I will make it trigger only once

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.

2 participants