fix(ffe-core): legg til font-family på linktext

[HeleneKassandra]

Beskrivelse

Legger til font-family på LinkText

Motivasjon og kontekst

Nå har ikke LinkText satt font-family, så i noen tilfeller så får du default font når du prøver å bruke linktext
fiksen gjør det sånn at man alltid får sparebank1 fonten.

Testing

Kjørt opp lokalt

[github-actions]

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1780.westeurope.2.azurestaticapps.net

[antidecaf]

Tenker i utgangspunktet at LinkText er et inline-level element som bør arve typografi fra parent? Ellers blir det nødvendig med overstyring hvis man skal lenke fra tekst som av en eller annen grunn har en annen font enn defaulten. I vår styling har f.eks .ffe-strong-text, .ffe-em-text, .ffe-pre-text og headinger egne fonter. Vet ikke hvor ofte disse inneholder lenker, sikkert ikke særlig ofte, men det er i alle fall et potensielt issue å ha i bakhodet.

[HeleneKassandra]

Tenker i utgangspunktet at LinkText er et inline-level element som bør arve typografi fra parent? Ellers blir det nødvendig med overstyring hvis man skal lenke fra tekst som av en eller annen grunn har en annen font enn defaulten. I vår styling har f.eks .ffe-strong-text, .ffe-em-text, .ffe-pre-text og headinger egne fonter. Vet ikke hvor ofte disse inneholder lenker, sikkert ikke særlig ofte, men det er i alle fall et potensielt issue å ha i bakhodet.

Ja, skjønner forsåvidt den, men i vårt tilfelle så fører det hvertfall til litt unødvendig komplisert HTML-syntaks.
Det scenarioet jeg kom borti er at man har en eller 2 lenker som står "alene", og derfor ikke arver fra noen parent (kan være det mangler noe som gjør at den ikke arver det fra body?). Sånn de har fikset det opp til nå er å wrappe linkene i en , men syntaks-messig gir det ingen mening å ha paragraph rundt, så det er kun for å hacke til fonten.

Eneste scenarioene man vil ha en annen font enn regular, er hvis det er h1-h6 eller strong text, så kan jeg f.eks legge inn "regular" fonten på ffe-link-text:not(h1,h2,...) osv? Er det en bedre løsning?

[antidecaf]

kan være det mangler noe som gjør at den ikke arver det fra body?

Ja, default typografi burde arves fra body i utgangspunktet. Den enkleste løsninga er nok å legge til class="ffe-body-text" der.

Eneste scenarioene man vil ha en annen font enn regular, er hvis det er h1-h6 eller strong text, så kan jeg f.eks legge inn "regular" fonten på ffe-link-text:not(h1,h2,...) osv? Er det en bedre løsning?

Jeg tenker det fortsatt legger en litt unaturlig begrensning, i og med at det er et inline element som helst skal arve denne type styling i utgangspunktet. Scenarioene du nevner er riktignok de eneste aktuelle nå, men det kan bli problematisk om man for eksempel skal style blockquotes, lead-paragrafer, kort eller andre ting som kan inneholde lenker annerledes enn brødtekst, i en eller annen app på ett eller annet tidspunkt. :)