Article comments: the reply form and buttons are children of the <ul> of comments

Aperta
  • Qualcosa non va?
#696 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@guidol-ive ci sta già lavorando.

Dal 29/10/2026.

  • #700 di @guidol-ive — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
25/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
html, javascript, php
Ambito
accessibility, frontend

Direzione di ricerca

Start with peer_reviewed_article::buildDiscussionsHtml() and trace how it builds the discussion wrapper, comment list, reply form, and action button. Move the form and buttons to the wrapper after the list, then check rendered public comments and review discussions to confirm the <ul> contains only <li> elements; linked pull request #700 is already open.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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

The comments under an article are a <ul> with one <li> per comment. The form to reply, the button to send it, and in a review the button to close the discussion, are appended to that same <ul>:

<div class="public-comment-wrapper collapsed"><ul>
  <li class="first-comment">…</li>
  <form …>…</form>
  <a class="button" …>Reageer</a>
</ul></div>

A <ul> may only hold <li> (and script-supporting elements). Screen readers announce the list with a wrong number of items, and axe reports list (WCAG 1.3.1).

On a test copy of a site, axe found it on every page with public comments, about 950 pages.

peer_reviewed_article::buildDiscussionsHtml() (the part that renders each discussion) calls $commentList->createChild('p'), $replyForm->appendTo($commentList) and $commentList->appendChild($this->makeDiscussionActionButton(…)).

Fix: append these to the wrapper <div> of the discussion, after the list. No script depends on the form being inside the list; the stylesheet of the default theme only styles ul and li inside the wrapper.

🤖 Generated with Claude Code

Lingua principale
JavaScript
Stelle
5
Fork
9
Merge medio
18h 49m
PR unite (30g)
13

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di PlanBCode/hypha

Tutte le issue di PlanBCode/hypha

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.