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

Any plans on adding to npm? #44

Open
jakehm opened this issue Nov 27, 2016 · 4 comments
Open

Any plans on adding to npm? #44

jakehm opened this issue Nov 27, 2016 · 4 comments

Comments

@jakehm
Copy link

jakehm commented Nov 27, 2016

This is the best way to animated a marker on leaflet that I've found. It would be easier to implement if there was an npm package.

And this requires jquery right? Add that the to the readme at least.

@Abbe98
Copy link

Abbe98 commented Nov 27, 2016

No it does not require jQuery. What makes you think that?

@jakehm
Copy link
Author

jakehm commented Nov 27, 2016

Because he links to it in the index.html for some reason.

@Abbe98
Copy link

Abbe98 commented Nov 27, 2016

Yes it's used in the demo(demo.js) for loops etc. If you take a look at the plugin's source you won't find any dependency on jQuery.

@paullryan
Copy link
Contributor

All thats needed to publish this is changing the name in package.json to leaflet.animatedmarker it's required to be all lowercase.

paullryan added a commit to SimplyComplexCo/Leaflet.AnimatedMarker that referenced this issue Jan 26, 2017
This is to address openplans#44
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

3 participants