Skip to content

Conversation

sebasti
Copy link
Member

@sebasti sebasti commented Oct 29, 2026

Issue

Closes #64

Bygger på #63, som bygger på #62. Skal merges etter dem. Til da viser PR-en også commitene derfra.

Endringer

  • Apache (deploy/apache/t36-project2.conf):
    • Komprimerer GraphQL-svar (application/json og application/graphql-response+json) og SVG i tillegg til HTML, CSS og JS.
    • Cache-headere: immutable i ett år for JS og CSS med hash, no-cache for index.html, og en uke for bilder.
    • DeflateAlterETag NoChange, så uendrede filer gir 304.
  • Mørkt tema som følger prefers-color-scheme. Bare fargevariablene i tokens.css endres. Kontrasten er regnet ut og sjekket med axe.
  • Ingen layout-forskyvning: main er minst like høy som vinduet, så bunnteksten ikke hopper ned når en side er lastet (CLS 0,069 → 0).
  • docs/baerekraft.md med målinger før og etter, bunten fordelt på pakker, og det vi vurderte, men lot være (kodesplitting, Brotli).

Resultat i katalogen: første besøk 164,3 → 160,3 kB, nytt besøk 5,7 → 2,2 kB, én side fra GraphQL 5,0 → 1,4 kB.

Verdt å se på: DeflateAlterETag gjelder hele serveren, også prosjekt 1.

Testing

  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm test (298)
  • npm run build
  • Kontrollert i nettleser i lyst og mørkt tema, på 1280 og 375 px, bak lokal Apache med den nye konfigurasjonen
  • Apache-konfigurasjonen på VM-en (gjøres i [CONFIG] Deploy innlevering 3 til VM-en #68, se deploy/apache/README.md)

Sjekkliste

  • README eller docs/ er oppdatert hvis det trengs

Sebastian Ingebrigtsen and others added 23 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>
Co-Authored-By: birkjr <birkjr@stud.ntnu.no>
Co-Authored-By: birkjr <birkjr@stud.ntnu.no>
@sebasti sebasti requested a review from a team as a code owner October 29, 2026 21:49
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 også! Fint at du har sett på både ytelse og ressursbruk, og at du har dokumentert målingene før og etter endringene. Spesielt bra at du har fått redusert størrelsen på dataene som overføres. Ser ryddig ut, og ingen umiddelbare innvendinger fra min side!

@birkjr birkjr merged commit a4e4437 into dev Oct 30, 2026
@tordtst tordtst deleted the 64-feat-mål-og-forbedre-bærekraften-i-appen 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