setLatLng constrained documentation + expose LatLngZoom #11184
Conversation
tobrun
changed the title
| * | ||
| * @param latLng Target location to change to | ||
| */ | ||
| public void setLatLngZoom(@NonNull LatLng latLng, |
There was a problem hiding this comment.
I don't think we should add this. It's already possible to set center and zoom simultaneously with moveCamera -- there's no need to add to what is already a sprawling camera API. If anything, we should be removing individual setters such as setLatLng and setZoom in favor of the unified moveCamera.
There was a problem hiding this comment.
I initially implemented that api for having a more performant way of doing camera updates with Android SDK animators. After running benchmarks on both setups, the difference is neglectable:
thank you for flagging @jfirebaugh, I will work on removing the API.
There was a problem hiding this comment.
@tobrun I'd be great to sync the removal of exposed setters with mapbox/mapbox-plugins-android#259 and #11170 as right now exposed setters are the only way to manipulate camera without a loss of tracking mode.
Closes #10928