gmaps
gmaps copied to clipboard
Add changeMarker function to library
A couple of comments:
- This is the first pull request I've ever made (as well as the first open source contribution I've ever made). Hopefully the code style is acceptable and whatnot. Feel free to hit me up with any advice / constructive criticism.
- Disregard commit e48b967.
- Motivation for the fork and pull request: I've been using your truly awesome library for a project for my internship. I've needed to dynamically update markers while maintaining their order, so invoking removeMarker and addMarker just wouldn't do it (it changed the order of markers). To solve that, I wrote up a changeMarker method.
- There'd be lots of shared code between createMarker and changeMarker, so I refactored it into an initializeMarker helper function.
Hi, I'm checking your code and I think it might fail in some cases. I will post some comments in your code, but first you need to apply your changes in lib/gmaps.markers.js since gmaps.js is a generated file.
Alright, thanks for letting me know. I feel kind of silly for one, not reading the contribution guidelines, and two, not running your test suite. I'll take a look at your comments later tonight and possibly issue a fresh pull request with added examples.