material-components / material-components/material-components-android
[MaterialDatePicker] Make the class open*.
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Java
- Estrellas
- 17.4k
- Forks
- 3.2k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
**Is your feature request related to a problem? Please describe.**
I would suggest the class to be ~made public~ open, so we could attach the selection listener in a subclass and use "sharedViewModel" to deliver the selected date. At the moment in case of screen rotation, we need to detach the listeners and reattach when the new fragment is ready to avoid a memory leak.
**Describe the solution you'd like**
Make the class public.
**Describe alternatives you've considered**
Maybe another way to avoid subclassing it would be the old fashion OnInteractionListener on the parentFragment / activity.
We also happily accept [pull requests](https://github.com/material-components/material-components-android/pulls).
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 en el punto de entrada de MaterialDatePicker y revisa cómo se asocian actualmente los listeners de selección durante la recreación de fragmentos. Compara el enfoque de subclasificación solicitado con la alternativa propuesta de un listener de parentFragment o activity. Se considerará completado cuando la API compatible permita claramente el uso previsto del listener y de sharedViewModel sin requerir la eliminación manual del listener durante la rotación.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- android, java
- Área
- api, mobile
- 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