Pulse /telos/item renders a <main> inside the layout's <main>

Chiusa Adatta ai principianti
#2,361 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 7 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
88/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
next.js, react, typescript

Direzione di ricerca

Leggi LifeOS/install/LIFEOS/PULSE/Observability/src/app/telos/item/page.tsx e Observability/src/app/layout.tsx per verificare i landmark annidati. Modifica il wrapper della pagina e il relativo tag di chiusura come descritto, mantenendo la classe telos-item. Il lavoro è completato quando /telos/item ha un solo landmark main di primo livello e lo stile della pagina rimane intatto.

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

Descrizione

LifeOS/install/LIFEOS/PULSE/Observability/src/app/telos/item/page.tsx wraps the page in <main className="telos-item"> (line 217, closing at 274). The root layout, Observability/src/app/layout.tsx line 41, already wraps every page in <main>. So /telos/item ships two nested <main> landmarks.

Why it matters: a page should have one top-level <main> landmark. Screen readers announce two main regions, and axe-core's landmark-main-is-top-level rule covers this case.

Fix: change the page's wrapper from <main className="telos-item"> to <div className="telos-item"> (and its closing tag). The class carries the styling, so nothing else should change; I have not run the fix on a stage.

Found in a code review of the Pulse pages on a local install; the file is unchanged from upstream.

Lingua principale
TypeScript
Stelle
19.7k
Fork
2.5k
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

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 danielmiessler/LifeOS

Tutte le issue di danielmiessler/LifeOS

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.