Datepicker Memory Leak
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, jquery
- Área
- frontend
Línea de trabajo
Inspecciona ui/widgets/datepicker.js alrededor de la línea 163 y sigue la ruta de destrucción descrita en el issue. Revisa las pruebas unitarias que dependen de dpDiv y reproduce la secuencia hide-then-destroy. Se considera terminado cuando la UI de datepicker ya no se conserva después de la destrucción y se han abordado las pruebas afectadas y el comportamiento de uso documentado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Building off #2268, there is a problem when destroying the datepicker where it doesn’t completely get removed from memory. This pull request, https://github.com/jquery/jquery-ui/commit/817ce38, removes the instance reference but the actual UI reference is tied to
When the widget is destroyed, the datepicker UI element remains in memory preventing it from closing. https://github.com/jquery/jquery-ui/pull/2268 patches this issue by hiding the component, however dpDiv is still in memory. One solution could be to remove dpDiv from the instance and then set both _curInst = null and this.dpDiv = null.
https://github.com/jquery/jquery-ui/pull/2268/commits/5e4f1b86296c5a977abc7efaff671ea9004d4d37
However, many unit tests will break because they rely on dpDiv to still exist in memory, requiring the datepicker to be recreated for each test. There is also a problem that many developers rely on hiding the datepicker before destroying it $.datepicker("hide").destroy("destroy") which will fail because dpDiv will be null.
- Lenguaje dominante
- JavaScript
- Estrellas
- 11.3k
- Forks
- 5.2k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de jquery/jquery-ui
-
Selectmenu changes selection after its native select is disabled while openPosiblemente ocupada @Xapollon430 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Datepicker changes a natively disabled input if its calendar is openPosiblemente ocupada @Xapollon430 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
-
Implementing trusted-types CSPAbiertoBehavior shared with 1.12 Comp: Datepicker
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
-
Comp: Datepicker Needs info
Dificultad 3/5 1-2 días Aptitud para principiantes 53/100
-
CSS variable stripped from calls to jQuery .css()Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. AbiertoComp: Effects
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
Todos los issues de jquery/jquery-ui
Issues similares
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
coder/claudecode.nvim#588 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
digitalbazaar/http-client#60 ·
-
Ruby: The link in the Blocks lesson points to an external article that isn't at that address anymoreAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
TheOdinProject/curriculum#31529 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
topmonks/hlidac-shopu#3687 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ONLYOFFICE/DocumentServer#3912 · 1 comentario ·