Skip to content

Conversation

@gruble
Copy link
Contributor

@gruble gruble commented Jan 9, 2026

Fixes #698.
Vet saken er merket med "må diskuteres", så ta gjerne dette som et innspill til diskusjonen:)
Det beste hadde kanskje vært om designsystemet kunne distribueres med font-filene designsystemet trenger. Jeg har tidligere gjort et forsøk på dette uten å lykkes. Se #684, som senere ble rullet tilbake fordi @tomapedersen fikk problemer med noen av applikasjonene de forvalter.

Denne PR'en er en enkel fiks som gjør at vi slipper advarsler og unødvendige kall til cdnfonts.com i offline-applikasjoner.
Har fjernet denne fra global.css:

@import url('https://fonts.cdnfonts.com/css/source-sans-pro');

Dette gjør at alle som bruker designsystemet fra nå av må laste skrifttyper selv i sine applikasjoner.
Har beskrevet dette i sida "Importering av filer" under "For utviklere".

  • Dette er en BREAKING CHANGE. Jeg prøvde å ta med "BREAKING CHANGE" i PR-tittelen, men det likte ikke byggesystemet, så tar gjerne imot tips om hvordan vi får endret hoved-versjon automatisk.

og dokumentert hvordan man kan gjøre dette selv i applikasjonen.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-743.westeurope.5.azurestaticapps.net

@gruble gruble changed the title BREAKING CHANGE: Fjernet global import av skrifttyper fix: Fjernet global import av skrifttyper Jan 9, 2026
@gruble gruble changed the title fix: Fjernet global import av skrifttyper feature: Fjernet global import av skrifttyper Jan 9, 2026
@gruble gruble changed the title feature: Fjernet global import av skrifttyper feat: Fjernet global import av skrifttyper Jan 9, 2026
@amish1188
Copy link
Contributor

amish1188 commented Jan 14, 2026

Jeg tror vi venter med den til vi ferdig diskuterte hvordan vi distribuerer fonter og om vi bytter til variables fonts.

@tomapedersen
Copy link
Contributor

Vi var jo nokså nærme å få det til å fungere med "font inkludert i DS-kode", men det var noe med path som ble feil når man brukte include i et prosjekt. Føler det er bedre å gå litt videre den veien istedenfor, heller skrive om doc-site slik at den bruker samme path som andre prosjekter.

@gruble
Copy link
Contributor Author

gruble commented Jan 14, 2026

Vi var jo nokså nærme å få det til å fungere med "font inkludert i DS-kode", men det var noe med path som ble feil når man brukte include i et prosjekt. Føler det er bedre å gå litt videre den veien istedenfor, heller skrive om doc-site slik at den bruker samme path som andre prosjekter.

Er enig med deg. @amish1188 nevnte igår at vi kanskje kunne teste om det hjalp å importere fonter med <link> i html i stedet for @import i css?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VI trenger ikke hente fonter automatisk når vi pakker fontene med applikasjonen

4 participants