Comments are included in readable content on lesswrong.js
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 58/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- javascript
- Área
- content
Línea de trabajo
The report reproduces the problem on a LessWrong post and points to its comments id as a possible clue. Start by tracing how Readability selects and scores content in the DOM, then check existing tests for comment-heavy pages. Done means the extracted readable content excludes the lengthy comments while retaining the post itself; the issue names no test or source file.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
https://www.lesswrong.com/posts/GLpFovxZdwXYwmbkJ/failures-in-kindness
Each lesswrong.com post includes a lengthy comments section.
Readability js includes the comment in the prettified dom:
The DOM is made up of all divs 😢, so the only approach comes to my mind is to penalize the comments id
- Lenguaje dominante
- JavaScript
- Estrellas
- 11.5k
- Forks
- 740
- 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 mozilla/readability
-
Data tables are removed as "unlikely candidates" when a class name contains "header" (e.g. Wikipedia's plainrowheaders)Posiblemente ocupada @axchadxs la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
mozilla/readability#1036 · 1 comentario ·
-
JSDOMParser: Element is missing a lang accessorPosiblemente ocupada @josuesto la tomó hace 19 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 76/100
mozilla/readability#1026 ·
-
PHRASING_ELEMS missing <u>, <s>, and <bdi> elements causes paragraph splittingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
mozilla/readability#997 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
mozilla/readability#1031 ·
-
macworld.com rendering problemsAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
mozilla/readability#1029 ·
Todos los issues de mozilla/readability
Issues similares
-
S: triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Dome-Works/DOME#129 ·
Los mantenedores suelen responder en 1 día
-
Add: Exclusiv DIGI TV 3Abiertochannels:add check:passed
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 6 días
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
githubnext/gh-aw-workshop#5077 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 12 días