Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

MGLMapCamera interprets viewing distance incorrectly #12943

Closed
1ec5 opened this issue Sep 22, 2018 · 0 comments · Fixed by #12966
Closed

MGLMapCamera interprets viewing distance incorrectly #12943

1ec5 opened this issue Sep 22, 2018 · 0 comments · Fixed by #12966
Assignees
Labels
bug iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS MapKit parity For feature parity with MapKit on iOS or macOS navigation For the Mapbox Navigation SDK for Android or iOS or navigation use cases in general

Comments

@1ec5
Copy link
Contributor

1ec5 commented Sep 22, 2018

+[MGLMapCamera cameraLookingAtCenterCoordinate:fromDistance:pitch:heading:] is based on MapKit’s +[MKMapCamera cameraLookingAtCenterCoordinate:fromDistance:pitch:heading:]. But even though the distance parameter is documented as “the straight-line distance from the viewpoint to the centerCoordinate”, it actually controls the altitude parameter, which is the height the viewpoint above the ground plane. This significant discrepancy has apparently always been present in the MGLMapCamera implementation, since #2193. Fixing it will likely require a semver-major release, but in the meantime it might be desirable to add a correct initializer alongside the buggy one.

/ref #12938
/cc @mapbox/maps-ios @d-prukop

@1ec5 1ec5 added bug iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS navigation For the Mapbox Navigation SDK for Android or iOS or navigation use cases in general MapKit parity For feature parity with MapKit on iOS or macOS labels Sep 22, 2018
@1ec5 1ec5 self-assigned this Sep 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS MapKit parity For feature parity with MapKit on iOS or macOS navigation For the Mapbox Navigation SDK for Android or iOS or navigation use cases in general
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant