-[MGLMapView selectAnnotation:animated:] does not pan map to fit callout view

[1ec5]

When YES is passed into -[MKMapView selectAnnotation:animated:], MKMapView automatically pans so that the entire callout view fits in the viewport. Instead, -[MGLMapView selectAnnotation:animated:]’s animated parameter controls whether the callout view fades in.

A fix for this discrepancy may have to wait until we fix #3154 and #2618.

/cc @incanus @friedbunny

[1ec5]

Now that #6676 has landed, this is no longer blocked.

[1ec5]

This could just be a matter of calling -showAnnotation: within -selectAnnotation:animated:. However, we’d have to take care not to reenter -selectAnnotation:animated: and cause infinite recursion.

[1ec5]

-showAnnotation: puts the annotation in the center of the viewport, which for point annotations would lead to an undesirable wandering effect as the user selects various annotations. But we could craft a simple method that pans the map only enough to show the point annotation and its callout.

[1ec5]

#2600 (comment) illustrates what the fix for this issue might look like inside MGLMapView.

[julianrex]

Closing since this was addressed in #11442