developit / developit/undom

NodeList

Abierto
#34 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
JavaScript
Estrellas
690
Forks
25
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

I tried using this for a unit-test, and ran into some issues with e.g. `childNodes`, which in regular DOM is a [NodeList](https://developer.mozilla.org/en-US/docs/Web/API/NodeList) and not just an `Array`, which is missing, for example, the `keys()`, `values()` and `entries()` iterator methods.

I can foresee issues as well with e.g. `children` being a plain `Array` created by filtering `childNodes` - whereas, in regular DOM, mutating the `childNodes` of the parent that generated the `children` collection, would cause the iterator to reflect those changes.

Maybe this is "by design", favoring simplicity over correctness?

If so, that isn't clear from the "Project Goals" stated in the README right now - it doesn't seem to imply that this deviates from DOM in terms of more than features.

Lacking features is a matter of scope - but incompatibilities probably ought to be designated in the documentation?

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza con los Project Goals del README e inspecciona cómo se producen actualmente childNodes y children. Compara el comportamiento con los métodos keys(), values() y entries() de NodeList y con las expectativas de mutación/liveness indicadas. Para darlo por terminado, se requiere una decisión clara sobre el alcance de compatibilidad, ya sea con desviaciones documentadas o con un cambio de implementación acotado y pruebas.

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

Evaluación

Stack tecnológico
javascript
Área
web-dev
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.