openPopup() breaks setView() with animation

Scenario is the following very common setup: clicking in sidebar to center map on marker and open marker’s popup. What happens instead: map pans to wrong place and/or moves only till marker is in the map, but not in the center. It also happened that the popup got displayed in a totally different place than the marker on the screen.

Abstract code for trigger:

item_title_link.click(function() {
    map.panTo(layer.getLatLng(), {animate: true});
    layer.openPopup();
});

What happens:

1. [bug]: When clicking for the first time, the map doesn’t center on the marker, but pans until it is in the corner/edge of map. In this case, panBy is called twice.
2. [good]: When clicking for the second time, the map centers on the marker. In this case, panBy is only called once.

The reason as far I was able to debug is the following. The following paths are all fighting to move the screen and end up in broken results many times.

1. panTo -> setView -> _tryAnimatedPan -> panBy is always called. This is good and it is the required behaviour.
2. Layer.openPopup -> Map.openPopup -> setLatLng -> update -> _adjustPan -> panBy. This should not happen, as it destroys the real panBy movement. This path is not happening once the marker is visible on the screen (at the edges/corners), thus the second time the activation happens this path doesn’t run.

The only “fix” I found is to call setView with {reset: true} to completely disable animation.

Leaflet v1.0.0-beta.1

OS X 10.9.5, latest Chrome

[clemenslermen]

Have you tried to set the Popup {autoPan: false}? I think I had a similar problem. Markers which were pretty much at the edge of the screen didn’t center when a popup was bound to them. Setting this option solved the problem.

[hyperknot]

Just to confirm: OK in beta 2.