Skip to content

Conversation

sebasti
Copy link
Member

@sebasti sebasti commented Oct 29, 2026

Issue

Closes #63

Bygger på #62 og skal merges etter den. Til da viser PR-en også commitene derfra.

Endringer

Gjennomgang mot WCAG 2.1 AA med axe-core (samme motor som Lighthouse), tastatur, 320/640 px, tekstavstand og prefers-reduced-motion. axe fant ingen brudd. Fire funn fra den manuelle gjennomgangen er rettet:

  1. Fokus ved blaing (2.4.3): fokus ble stående på «Neste» nederst mens siden viste toppen. Nå går fokus til seksjonen «Treff» når bare siden endres.
  2. Kontrast på plassholdertekst (1.4.3): 3,5:1 i Firefox. Nå 6,3:1 i alle nettlesere.
  3. Fokus under lagring (2.4.3): disabled på knappen mens skjemaet lagres flyttet fokus til toppen av siden. Nå aria-disabled, og knappen beholder fokus.
  4. Fokus etter «Fjern alle filtre» og «Vis alle annonser» (2.4.3): fokus havnet på <body>. Nå filterpanelet eller treffene.

I tillegg:

  • src/accessibility.test.tsx kjører axe-core på alle sidene og tilstandene i npm test.
  • Global regel som slår av animasjoner med prefers-reduced-motion.
  • docs/tilgjengelighet.md med metode, resultater, funn og en sjekkliste for skjermlesertest.

Verdt å se på: axe-core er ny dev-avhengighet.

Testing

  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm test (298)
  • npm run build
  • Kontrollert i nettleser, også på 320 px og med bare tastatur
  • Skjermlesertest med VoiceOver eller NVDA, se sjekklisten i docs/tilgjengelighet.md

Sjekkliste

  • README eller docs/ er oppdatert hvis det trengs

Sebastian Ingebrigtsen and others added 19 commits October 29, 2026 22:01
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
…ser» forsvinner

Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
@sebasti sebasti requested a review from a team as a code owner October 29, 2026 21:32
Copy link
Member

@birkjr birkjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bra jobbet med denne! Ser ut som du har vært grundig, spesielt med testingen og dokumentasjonen. Fint at du har fått på plass automatiserte tester også, så det blir enklere å fange opp tilgjengelighetsproblemer videre. Ingen spesielle innvendinger fra min side, ser ryddig ut!

@birkjr birkjr merged commit 6a5d49b into dev Oct 30, 2026
@tordtst tordtst deleted the 63-feat-gå-gjennom-og-forbedre-tilgjengeligheten branch October 30, 2026 13:26
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants