> I am in favour of aborting the movement on second click (maybe then when zoom out level has been reached?) and immediately continuing in the new direction.
The only thing we can easily do is discard the current animation and plan a new one starting from the current viewpoint, but targetting the original zoom. I had that implemented at some point and felt the sudden change in direction was not nice. I can bring it back if desired though.
> Maybe the movement speed needs to depend on how far apart the 2 points are?
I experimented a long time with the movement settings and the various heuristics involved. I also tried making the time dependent on the distance between points and felt this was sluggish for large distances. I made the code intentionally easy to tune (everything is done in mapview.cc), but I do not have more to contribute there. Somebody else with a fresh palette for these animations must look into tweaking the parameters.
> I am in favour of aborting the movement on second click (maybe then when zoom out level has been reached?) and immediately continuing in the new direction.
The only thing we can easily do is discard the current animation and plan a new one starting from the current viewpoint, but targetting the original zoom. I had that implemented at some point and felt the sudden change in direction was not nice. I can bring it back if desired though.
> Maybe the movement speed needs to depend on how far apart the 2 points are?
I experimented a long time with the movement settings and the various heuristics involved. I also tried making the time dependent on the distance between points and felt this was sluggish for large distances. I made the code intentionally easy to tune (everything is done in mapview.cc), but I do not have more to contribute there. Somebody else with a fresh palette for these animations must look into tweaking the parameters.