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

js events #1

Closed
maxschulmeister opened this issue Mar 19, 2017 · 3 comments
Closed

js events #1

maxschulmeister opened this issue Mar 19, 2017 · 3 comments

Comments

@maxschulmeister
Copy link

thanks for the directive, it works as expected!
However, i think it would be great to also fire some events to make it accessible not only through css.
What do you think about that, any ideas / plans?

cheers!

@weotch
Copy link
Member

weotch commented Mar 20, 2017

Huh, interesting. I guess, as a directive, it would be possible to fire arbitrary JS events (like as opposed to Vue events).

However, when I've needed to do more complicated things, I use this other, related, package we made: https://github.com/BKWLD/vue-in-viewport-mixin. Have you seen it? It might work for your needs.

@maxschulmeister
Copy link
Author

mhm the mixin works for components only, so unless you wrap every single el in a comp it's not gonna do the job right?

Gonna go with standalone scrollMonitor, thank you for building the mixin and directive though!

@weotch
Copy link
Member

weotch commented Mar 20, 2017

Yeah, that is true.

Sure, you bet!

@weotch weotch closed this as completed Mar 20, 2017
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

No branches or pull requests

2 participants