plotly / plotly/plotly.R

Mapbox "fitBounds" handling

Abierto
#2,255 0 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
R
Estrellas
2.7k
Forks
641
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

As per https://github.com/plotly/plotly.R/blob/9ee548023bba6d68dd953f46485b2e611026766b/inst/htmlwidgets/plotly.js#L206-L220

I believe the if(!args) check should be changed (or the parameter for args should default to null or some other calculated default, not {}) as it currently causes an "e is undefined" error in mapbox (where e is the 2nd parameter).

I've got around it by explicitly providing a mapbox = list(_fitBounds = list(c(xmin, ymin), c(xmax, ymax)), options = list()) entry in my plotly::layout call.

This also serves as a reminder that the fitBounds doesn't seem to work for restyle either.

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 por las líneas 206–220 de inst/htmlwidgets/plotly.js, centrándote en el valor predeterminado del argumento fitBounds y en la llamada a Mapbox que produce el error del segundo parámetro undefined. Reproduce el problema con y sin una lista explícita de opciones de mapbox y, después, verifica que fitBounds también funciona durante restyle.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript, r
Área
data-visualization, frontend
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.