FlyTo speed geocoder result

[martinrisseeuw]

Just a quick question, is it possible to adjust the flyto speed of the geocoder since the FlyTo option is a boolean.

[tristen]

That was a bit of an oversight on my part. flyTo should pass on the options available in https://www.mapbox.com/mapbox-gl-js/api/.

---

As an interim solution you can set flyTo: false, listen for the result and write your own map.flyTo.

var geocoder = new mapboxgl.Geocoder({
  flyTo: false
});

geocoder.on('result', function(e) {
  var result = e.result;
  map.flyTo({
    // Pass result and custom animation
  });
});

[martinrisseeuw]

@tristen Thanks for the reply, seems to work now. Only problem that I have now is that with the standard geocoder it zooms further if the address is more specific. How should I add this to the custom animation? Do you have any suggestions for this.

[tristen]

Well this is travelling further down a rabbit hole but you can infer from the source code that it actually fits to bounds if the address is less specific and zooms to fixed zoom if it's more specific.

[martinrisseeuw]

@tristen Great, thanks for the help.

[tristen]

Oh! we should keep this open. I think we should totally pass flyTo options over!

[andrewharvey]

Oh! we should keep this open. I think we should totally pass flyTo options over!

Agreed, for my use case it would nice to reset the pitch to 0 by simply passing it as an option to flyTo rather than rolling my own on result event handler.

[lbud]

This would be great — there's a lot moving in the Mapbox GL Camera API right now (mapbox/mapbox-gl-js#2801); we should do this after that lands (as it is, mapbox-gl-geocoder uses different camera methods depending on the result).

[andrewharvey]

With #214 in mind, slated for v4.

I think we should try to decide if those flyTo options are also applied when fitting the map to a geocode bbox.

mapbox-gl-geocoder uses fitBounds when a bbox is found, and fitBounds also supports GL JS CameraOptions and AnimationOptions https://docs.mapbox.com/mapbox-gl-js/api/#map#fitbounds

cc @scottsfarley93

[scottsfarley93]

👍 didn't realize these properties were available on the fitBounds method was well. I think we should pass any custom camera/animation options to the fitBounds calls as well.

[scottsfarley93]

This was done in a combination of #227 and #214. It will be available once version 4 is released.