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

Allow resetting of the animation #12

Closed
wants to merge 5 commits into from

Conversation

dgorissen
Copy link

No description provided.

@atogle
Copy link
Member

atogle commented Feb 5, 2014

@dgorissen My bad for being so slow on this. Do you have an example of this running I can take a look at? That would be really helpful.

@dgorissen
Copy link
Author

Not publicly available unfortunately. Also that project has now been finished and shelved but will try to dig up some example when I can find the time.

@dkniffin
Copy link
Contributor

I got a version of this running on my setup. Unfortunately I don't have it publicly available. Effectively, if I understand it correctly, what this does is after zooming in or out, the marker moves to the end of the animation.

It's semi-related to my issue here: #7

@gam6itko
Copy link

use a http://jsfiddle.net/


// Move to the next vertex
this.setLatLng(this._latlngs[this._i]);
this._i++;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should not do it
Marker is not moving

@atogle atogle closed this Feb 18, 2014
@gam6itko
Copy link

Why you deny this PullRequest ?

@gam6itko gam6itko mentioned this pull request Feb 19, 2014
@atogle
Copy link
Member

atogle commented Feb 19, 2014

@gam6itko Because you said "you should not do it Marker is not moving" and I don't have the bandwidth to fix it. I'm happy to look at an update or new pull request with a working example.

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.

None yet

4 participants