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

Added isolatedElement as parameter to scroll functions / event handlers to prevent multiple scroll elements from firing #58

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mjheller
Copy link
Contributor

With multiple scroll elements on same page, broadcasting scroll events will cause all scrolls to change

…rs to prevent multiple scroll elements from firing

With multiple scroll elements in same page, broadcasting scroll event will trigger all scrolls
Added isolatedElement as parameter to scroll functions / event handle…
@echa
Copy link
Owner

echa commented Apr 26, 2017

Thanks. Can you please make a short working example so I understand the requirement better. I wonder if it's cleaner to use the already existing id feature here. At configuration time you would set the scrollable's id and you would use the same id when sending events. Each scrollable would filter events and only process those events directed to it's own id.

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