Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ffe-core): utvide breakpoint for typografi på tablets #2073

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

tuva-odegard
Copy link
Contributor

fixes #1403

Beskrivelse

Breakpointet for tablets var litt for lite til å også tilrettelegge for landscape mode. Før denne committen fikk vi mindre skriftstørrelse på h1, h2 og h3 ved breakpoint-md (768px), men i følge microsoft kan landscape tablets være opp mot 1008px.

Har nå satt det til breakpoint-lg (1024px), så litt høyere enn microsoft sin standard, men tenker det er bedre at vi er konsistente i bruken av breakpoints internt. Så, fra 1023px og nedover så er skriftstørrelsen for h1, h2 og h3 litt mindre. Håper at det blir bedre for flere.

Vi nevner ikke typografi-breakpoints i dokumentasjonen så det trengs ikke oppdateres noe i designssystem-docs.

Testing

https://sparebank1.github.io/designsystem/typography_Heading sjekk på forskjellige skjermsørrelser.

@tuva-odegard tuva-odegard requested a review from a team as a code owner June 7, 2024 12:08
@pethel
Copy link
Contributor

pethel commented Jun 7, 2024

Jag føler @antidecaf må ta denne. Jag kjenner en så lenge ikke typografin gott nok før att si hvordan det henger sammen og vill påvreka ting det her.

@tuva-odegard tuva-odegard changed the title fix(ffe-core): utvide brakpoint for typografi på tablets fix(ffe-core): utvide breakpoint for typografi på tablets Jun 10, 2024
Copy link

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

@tuva-odegard tuva-odegard merged commit c7f93f6 into develop Jun 10, 2024
3 checks passed
@antidecaf antidecaf deleted the 1403-table-breakpoints branch June 10, 2024 13:40
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.

Endre breakpoints for typografi på tablet
3 participants