python-visualization / python-visualization/folium
Add template parameters to JsCode objects
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 7.4k
- Forks
- 2.3k
- Merge medio
- 17 h 22 min
- PR fusionados (30 d)
- 11
Descripción
Is your feature request related to a problem? Please describe.
I'd like to be able to refer to other folium /leaflet objects inside a JsCode object. So far I have been using string interpolation inside the body of the JsCode object.
Describe the solution you'd like
I'd like to add parameters to the JsCode __init__ method, together with a basic string templating solution. For the string templating I'd prefer to use the good-old %(name)s format.
Describe alternatives you've considered
I considered using the newer formatting templates such as used in python format, or Jinja templates. However, due to brace heavy nature of javascript, I think a completely new variable marker will lead to more readable code.
Implementation
folium is maintained by volunteers. Can you help make a PR if we want to implement this feature?
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
Comienza leyendo el punto de entrada JsCode init y los usos actuales de la interpolación de cadenas en los cuerpos de JsCode. Compara el comportamiento solicitado de los parámetros %(name)s con el enfoque existente de generación de JavaScript. Se considerará terminado cuando JsCode acepte parámetros y los renderice usando el formato básico de plantillas propuesto, con cobertura para el nuevo comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, python
- Área
- data-visualization, web-dev
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100