Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

MapboxNavigationActivity camera not hooked automatically Maps 8.1.0 #1992

Closed
Guardiola31337 opened this issue Jul 8, 2019 · 1 comment 路 Fixed by #1995
Closed

MapboxNavigationActivity camera not hooked automatically Maps 8.1.0 #1992

Guardiola31337 opened this issue Jul 8, 2019 · 1 comment 路 Fixed by #1995
Assignees
Labels
bug Defect to be fixed.
Milestone

Comments

@Guardiola31337
Copy link
Contributor

When integrating #1890 and testing the test app NavigationLauncherActivity (MapboxNavigationActivity) using Maps SDK v8.1.0 we've noticed that the camera is not hooked (tracking) automatically anymore 馃憖

mapbox_navigation_activity_camera_not_hooked_8_1_0

This is the upstream PR causing the regression on our side mapbox/mapbox-gl-native#14745

We're probably scheduling a MapboxMap animation after setting the tracking mode. We can start debugging by placing a breakpoint here https://github.com/mapbox/mapbox-gl-native/blob/4c896cbb31fcb929b0e57c7bd02e519c267a1be1/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponent.java#L1561
this should give us a good idea about where do those updates come from.

cc @andrlee

@DejanRistic
Copy link

DejanRistic commented Jul 15, 2019

@Guardiola31337 I'm actually still noticing the issue on 0.41.0 and 8.1.0, but manually invoking callOnClick() on the recenter button after start navigation seems be a good workaround for me. It could be a result of something else we're doing that's causing it to still be an issue. Just wanted to toss it out in case others are still having issues with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Defect to be fixed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants