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

Flickers marker, when other MarkerView animated #6422

Closed
klblk opened this issue Sep 22, 2016 · 6 comments
Closed

Flickers marker, when other MarkerView animated #6422

klblk opened this issue Sep 22, 2016 · 6 comments
Labels
Android Mapbox Maps SDK for Android

Comments

@klblk
Copy link

klblk commented Sep 22, 2016

Platform: Android (Nexus 4, Android 5.1)
Mapbox SDK version: 4.2.0-beta3

Marker added through MarkerOptions flickers, when other MarkerView animated.

Steps to trigger behavior

  1. https://github.com/mapbox/mapbox-android-demo/blob/master/MapboxAndroidDemo/src/main/java/com/mapbox/mapboxandroiddemo/labs/OffRouteActivity.java

change: destinationMarker = map.addMarker(new MarkerViewOptions().position(point));

to: destinationMarker = map.addMarker(new MarkerOptions().position(point));

4.2.0-beta2 does not have this problem.

@cammace cammace added the Android Mapbox Maps SDK for Android label Sep 22, 2016
@cammace
Copy link
Contributor

cammace commented Sep 22, 2016

ezgif com-video-to-gif 6

Was able to reproduce on a Nexus 5x and will look more into this. Is there a paticuiliar reason for switching the detinationMarker? I recommend using Marker Views in almost all cases now unless you have an abundant amount of markers on your map and don't need the additional customization that comes with marker views.

cc: @tobrun @zugaldia

@klblk
Copy link
Author

klblk commented Sep 23, 2016

I add over 1000 static markers through MapboxMap.addMarkers() in my project. MapboxMap.addMarkerViews() slowly and map freezin (with 1000 MarkerView).

@tobrun
Copy link
Member

tobrun commented Sep 23, 2016

Going to do some debugging to see what the cause is of this issue.

UPDATE:
Was able to reproduce it in test app:

ezgif com-video-to-gif 61

Going to git bisect this issue

@tobrun
Copy link
Member

tobrun commented Sep 23, 2016

I have tried bisecting this 3 times but always ending up on a different commit 😅

@cammace
Copy link
Contributor

cammace commented Nov 16, 2016

Not seeing this issue anymore in the Android SDK.

ezgif com-video-to-gif 3

Closing

@cammace cammace closed this as completed Nov 16, 2016
@klblk
Copy link
Author

klblk commented Nov 17, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

No branches or pull requests

3 participants