googlemaps / googlemaps/android-maps-compose
Replace most GoogleMap() rememberUpdatedState() usages with single State object and encapsulate Subcomposition functionality
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Kotlin
- Estrellas
- 1.3k
- Forks
- 181
- Merge medio
- 2 d 23 h
- PR fusionados (30 d)
- 18
Descripción
The GoogleMap() implementation uses a gazillion individual rememberUpdatedState() calls. This is unwieldy and likely contributed to #481.
Instead, a single State object encapsulating the individual state properties can be used and passed as a parameter. This pattern is in compliance with Google Compose API guidelines.
The subcomposition should be made a separate composable function to prevent accidental access to local variables and better encapsulate subcomposition functionality.
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 localizando la implementación de GoogleMap() e inventariando sus usos de rememberUpdatedState() y el código de subcomposition. Revisa el patrón de las directrices de Google Compose API y, después, separa la subcomposition y reemplaza la gestión individual del estado por un único objeto State encapsulador; el trabajo estará terminado cuando la implementación esté encapsulada sin cambiar el comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- android, kotlin
- Área
- mobile
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100