17 docs dokumentere løsning testing og ki bruk i readme #32
+1,110
−860
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lser - Bytt aksentfargen i lys modus fra #aa3bff til #8b2fd9. Den gamle ga 3,8:1 på aksentflaten, under kravet på 4,5:1 i WCAG AA. Den nye gir 5,1:1 på aksentflaten og 6,0:1 på hvit. - Sett grunnstørrelsen i prosent og overskriftene i rem, slik at teksten følger nettleserens innstilling for tekststørrelse. - Gi h1 egen linjehøyde, så linjene ikke overlapper når overskriften brytes. Margene er justert, så innholdet flytter seg høyst 3,8 px. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Nettleseren gjenbrukte bildeelementet og viste flagget til forrige land
til det nye var lastet, for eksempel «Brazil» med Bangladeshs flagg.
key={flagUrl} gir et nytt bildeelement for hvert flagg. Testen feiler
uten rettelsen.
Refs #17
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- «Hopp til landkortet» er første tabulatorstopp, så tastaturbrukere slipper å tabbe gjennom 25 land. Målet får fokus og viser en ramme ved tastaturbruk. - Bunnteksten oppgir countries.dev, GeoNames (CC BY 4.0) og Flagpedia.net. CC BY krever at kilden oppgis der dataene vises. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- SortControl og FavoritesFilter: props og brukerinteraksjon. - useSortOrder, useShowOnlyFavorites og favoritesFilterStorage: startverdi, lagring og blokkert lagring. - App: sorteringen huskes etter ny montering, valgt land beholdes ved ny sortering, data vises videre når en ny henting feiler, snarveien og kildehenvisningen. - CountryNavigation med bare ett land. Antall tester går fra 52 til 70. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
hero.png, react.svg, vite.svg og icons.svg fra Vite-malen var ikke i bruk. Favikonet var Vite-logoen og er byttet ut med en jordklode. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
eslint-config-prettier må ligge sist for å slå av regler i konfigurasjonene før den som kan komme i konflikt med Prettier. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- Beskriv navn på branches og at de lages fra issuen. - Skriv Refs #N i commit-meldingene, og før medforfattere og KI-verktøy med Co-authored-by på egne linjer. - Legg til regler for pull requests med Closes #N og godkjenning. - Legg npm run test til i Definition of Done. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- Skriv om README etter akseptansekriteriene i #17: funksjoner mot kravene, installasjon og kommandoer, datakilde og kildehenvisninger, TanStack Query, state og props med kodehenvisninger, Web Storage, responsivt design, tilgjengelighet, testing, kjente begrensninger, arbeidsflyt og KI-bruk med Cursor og erfaringer. - Samle alle gjennomførte tester i docs/testing.md og fjern utdatert tekst, blant annet en manuell test som fortsatt sto som «gjenstår». - Legg til publiseringsveiledning i docs/publisering.md. - Oppdater docs/accessibility.md med snarvei, kontrast, tekststørrelse og kjente begrensninger. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- Skriv tydelig at KI er brukt som sparringspartner og ikke som fasit, og list opp hva gruppen har gjort selv. - Legg til eksempler på at KI tok feil: feil påstander om koden i et README-utkast, to falske feil i de skriptede nettlesertestene, og nedtrekkslisten gruppen byttet ut med en sidemeny. - Vis i testavsnittene hva gruppen testet selv underveis og på egne enheter, og hva KI bidro med. Refs #17 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
19 tasks
tordtst
approved these changes
Sep 18, 2026
Sign in
to join this conversation on GitHub.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.