mapbox / mapbox/mapbox-plugins-android

Should `MarkerViewManager` implement a more 'reactive' camera listener ?

Abierto
#1,115 2 comentarios 2 reacciones 0 asignados Ver en GitHub

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:
![sluggish](https://i.imgur.com/MfsZelV.gif)

I tried replacing `MapView.OnCameraDidChangeListener` by `MapView.OnCameraIsChangeingListener` and had no more updates ?!

I then tried using `MapboxMap.OnCameraMoveListener` with better results:
![okish](https://i.imgur.com/ZNweZfQ.gif)
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

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.