Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Article: the heading of the comments comes after the comments and the form

Abierto
  • ¿Algo no está bien?
#628 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@guidol-ive ya está trabajando en esto.

Desde el 28/10/2026.

  • #630 de @guidol-ive — abierto

Evaluación

Dificultad
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
1/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
php
Área
frontend

Línea de trabajo

Read system/datatypes/peer_reviewed_article.php and inspect createDiscussionsDomElement(), especially where it appends the art-comments and art-review-comments headings. Move the heading before the comment list, as described in the issue; done means both headings appear before their lists and forms. Pull request #630 is already open against this issue.

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

Descripción

Component: User interface Impact: Low Level: Easy Pagetype: Peer reviewed article Type: Bug Value: Visitors

On a published article the heading Reacties (art-comments) comes after the comments, after the comment form and after the button, at the very end of the comments block. Without comments, the list item "nog geen reacties" therefore sits directly under the sources, as if it were one of them, and the heading follows below the form. The same happens to the heading Reacties van reviewers (art-review-comments) during the review.

Place. system/datatypes/peer_reviewed_article.php, createDiscussionsDomElement() (main a019475, lines 1640-1641): the <h2> is added with appendHtml() as the last step, after the list and the form.

Since when. Hypha 2018 (1fbcb5e) appended the heading first. The cleanup of 06584a6 (2019) moved it to the end of the function, and d333a73 (2025) kept that order when it replaced append() by appendHtml().

Evidence. Seen on a published article of the fictional test site, as a visitor, on an iPhone (iOS 27, Safari), on Android 15 (Chrome) and on the desktop, PHP 8.2 and 8.5:

<div class="sources"><h2>bronnen</h2>…</div>
<div class="public-comments-wrapper"><ul><li>nog geen reacties</li></ul> … <form …> … <h2>reacties</h2></div>

Fix. Add the heading at the start of createDiscussionsDomElement(), before the list. A pull request follows.

Found while building the Markdown editor (#520).

🤖 Generated with Claude Code

Lenguaje dominante
JavaScript
Estrellas
5
Forks
9
Merge medio
19 h 10 min
PR fusionados (30 d)
12

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de PlanBCode/hypha

Todos los issues de PlanBCode/hypha

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.