Persist user tracking mode despite programmatic camera changes

[frederoni]

Fixes #7789

The stuttering effect is not as noticeable as I first thought. Concurrent animations (#3625) might still be a blocker though.

The transition is more or less smooth depending on the timing of the subsequent location update.

@bsudekum 👀

[mention-bot]

@frederoni, thanks for your PR! By analyzing this pull request, we identified @1ec5, @incanus and @boundsj to be potential reviewers.

[bsudekum]

@frederoni nice! Are there any other camera changing function that this should also be removed from?

I like @1ec5 summed it up:

In other words, we should avoid kicking the user out of user tracking mode due to a programmatic camera change, but still kick the user out of user tracking mode in response to a user gesture. This would fix #1490 as well.

[frederoni]

All camera updating functions are chained so this change will impact all of them.

In other words, we should avoid kicking the user out of user tracking mode due to a programmatic camera change, but still kick the user out of user tracking mode in response to a user gesture. This would fix #1490 as well.

The change in this PR would fix that.

[1ec5]

What about -flyToCamera:withDuration:peakAltitude:completionHandler:?

[frederoni]

@1ec5 Good catch. Fixed in 8f79fea

[1ec5]

Please add a changelog entry to the iOS changelog.