search

Altinn / app-frontend-react

Altinn application React frontend
BSD 3-Clause "New" or "Revised" License
16 stars 24 forks source link

Hjelpeknapp tekst, feil font #2002

Closed DanRJ closed 1 month ago

DanRJ commented 1 month ago

Description of the bug

Hjelpeknapp tekst har font family Times New Roman selv om det står Inter i htmlen, jeg funderer på om Inter aldri blir lastet inn. image

Steps To Reproduce

Åpne en hjelpeknapp og se på teksten, evt se bildet ovenfor

Additional Information

No response

olemartinorg commented 1 month ago

Reprodusert lokalt, jeg får samme problem. Det ser ut til at designsystemet tvinger inter-fonten her, men i apps har vi ikke gått over til den fonten (enda). Dumt om font blir overstyrt fra designsystemet sin side.

Det kan også nevnes at denne feilen bør brukes som en anledning til å instruere Cypress til å klikke åpen en hjelpetekst i minst ett av skjermbildene som tas med Percy, slik at vi hadde fått et varsel om slike feil som dette (bør nok kombineres med et klikk på en lang/flerlinje-hjelpetekst, slik at vi får med hvordan det ser ut i samme slengen.

Magnusrm commented 1 month ago

Will get fixed by: https://github.com/Altinn/app-frontend-react/pull/2009

olemartinorg commented 1 month ago

Great! Does that mean that the issue I created in the design system (https://github.com/Altinn/altinn-design-system/issues/339) is no longer an issue there, then? Or are we overriding the font in app-frontend?

Remember the previous point about creating Percy tests for this as well. :pray:

Magnusrm commented 1 month ago

It seems it is no longer an issue 😊 Tested with the same app-frontend app version. And the new DS version fixes it. I'll remember to add a percy test! 👍