mapbox / mapbox/mapbox-plugins-android
Should `MarkerViewManager` implement a more 'reactive' camera listener ?
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Java
- Estrellas
- 242
- Forks
- 116
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Hello,
The `MarkerViewManager` implements the `MapView.OnCameraDidChangeListener` so updating a `MarkerView` is only updated after the camera did change, and not while the camera is changing.
The consequence of that is some sluggish `MarkerView` re-positioning after a camera move, as shown here:

I tried replacing `MapView.OnCameraDidChangeListener` by `MapView.OnCameraIsChangeingListener` and had no more updates ?!
I then tried using `MapboxMap.OnCameraMoveListener` with better results:

https://imgur.com/ZNweZfQ
GPU usage is a tad high, that's why I'm not sending a PR.
Should `MarkerViewManager` implements a more 'reactive' camera listener ?
PS: on the above gifs, I'm using react-native with:
- https://github.com/react-native-mapbox-gl/maps
- and https://github.com/react-native-community/lottie-react-native for the animated orange markers
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza localizando MarkerViewManager y su implementación de MapView.OnCameraDidChangeListener; después, compara su comportamiento con el de los listeners de cámara mencionados en el informe. Reproduce el reposicionamiento lento de los marcadores con la configuración enlazada de React Native y Lottie; se considerará terminado cuando las actualizaciones sean más fluidas sin un coste inaceptable para la GPU.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- android, java
- Área
- mobile
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100