Doors Phase 3: author profile links and headings, and two small review items

Offen
#983 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
55/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
html
Bereich
accessibility

Rechercherichtung

Read layouts/authors/list.html and the Phase 3 redesign context; compare the social links with layouts/partials/social_links.html and the fix in #975. Check keyboard access to contents links under the cookie banner at 1440 px and heading wrapping at 390 px with text at 200%. Done when the profile links have accessible names, heading order is corrected, and the two review issues are addressed in the redesign.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

[!NOTE]
🤖 Claude Opus 5.5 responding on behalf of Lukas Wallrich

Found by the independent review of the Doors preview. The first two are also on main, in the retained author profile template layouts/authors/list.html, for example at /authors/lukas-wallrich/.

  • Social links can't be reached by screen readers: the icons sit in ul.network-icon[aria-hidden=true], and the links have no names. Article author cards were fixed in #975 through layouts/partials/social_links.html, which this template doesn't use.
  • Heading order: the role and organisations are h3 directly under the h1.

Phase 3 redraws this template, so both can be fixed there.

Smaller, from the same review:

  • Contents links under the cookie banner: at 1440 px, the last link of a long contents card can sit behind the banner when reached by keyboard, until the banner is dismissed. The banner is restyled in Phase 3.
  • A single letter left on a line: at 390 px with text simulated at 200%, the Resources heading breaks as "Teachin" and "g". There is no overflow, so this is only polish.
Vorherrschende Sprache
HTML
Sterne
42
Forks
33
Ø Merge
16 Std. 3 Min.
Gemergte PRs (30 T.)
121

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus forrtproject/forrtproject.github.io

Alle Issues in forrtproject/forrtproject.github.io

Ähnliche Issues

Weitere Issues zu Accessibility

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.