-
Notifications
You must be signed in to change notification settings - Fork 314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow overriding the location puck's location #324
Comments
This is being tracked upstream in mapbox/mapbox-gl-native#6867. |
Currently, it's not possible to set up location simulation to override the puck's location like in the example app using just Core Navigation, as the example app makes use of this category on MGLMapView (https://github.com/mapbox/mapbox-navigation-ios/blob/master/MapboxNavigation/MGLMapView%2BMGLNavigationAdditions.h). |
#402 moves that category to the example application proper, since it will no longer be used by the navigation SDK. Even now, we could eliminate the category in favor of |
#323 (comment) describes a way to precisely customize the location of the user puck on screen that should tide us over until #402 lands. |
#402, which we hope to land next week after some testing, will reimplement course tracking without hackily overriding Note that this ticket only covers the navigation SDK’s own ability to control the geographic location of the user puck. It has no direct impact on developers using this SDK. #323 is about allowing the developer to control the puck’s position on screen. |
Closing now that #402 is in. |
This is an upstream issue in mapbox-gl-native (@1ec5 I can't recall the ticket), but based on the release cycles of mapbox-gl-native, we should consider adding a workaround to the navigation SDK that allows developers to customize the location used to display the maps location puck while in course tracking mode.
The example app's
CustomViewController
gets around this by disabling the mapView's location and heading updates and overriding it's internal locationManager delegate callback 😬:cc @bsudekum
The text was updated successfully, but these errors were encountered: