Skip to content

Conversation

@KittyGiraudel
Copy link

Hey there,

This should remove the jQuery dependency and solve #6. Be sure to test it though. :)

@sankalpsans
Copy link

Hey, yes. I will open up a PR once this is ready and stable! :)

@sankalpsans
Copy link

#8 Here we go.

@arbelh
Copy link
Owner

arbelh commented Jul 11, 2016

@hugogiraudel
Hey there,
Thanks for the contribution.
This has worked fine so far in my tests, but I will do further tests. It'd be better to add a proper DOM ready trigger to it to prevent early execution if the target elements are not loaded yet. I will release it as V3 as this is a major change and will mention you in credits!

@sankalpsans
Copy link

@arbelh Glad it works! Thanks for accepting to release it! :)

@sankalpsans
Copy link

@arbelh Hey, been a long time. Any timeline on when this is planned to be released?

@sankalpsans
Copy link

Hey @arbelh any update on this? Is there something more that is expected from this feature?

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.

3 participants