-
Notifications
You must be signed in to change notification settings - Fork 11
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
Ideas for other modifiers #85
Comments
How about a simple functional one like |
I think that's a great idea. |
Yeah it'll just wrap around https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollTo. We can have another one for |
I like it. Do you want to just make a separate issue for each modifier you want to propose? Then you can outline what the external API would look like, as well as the high level implementation details of which browser APIs you would leverage. That way, we can go through each of them and proceed from there, using each issue as the task to close when someone implements the deliverable. |
Yeah we can do that |
Something like |
What other scroll-based modifiers would you like to see added?
The obvious one is
did-scroll
- what other contextual interactions would be useful? Maybe modifier that is tailored to generating impressions? e.g. when an element scrolls into the visible viewport, it calls the handler to note that an impression was registered, configurable with different % visibility thresholds.The text was updated successfully, but these errors were encountered: