Page maxWidth-token #2572
Page maxWidth-token #2572
Conversation
🦋 Changeset detectedLatest commit: 4a2c44d The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Storybook demo4b5e3fd90 | 61 komponenter | 400 stories |
|
Hva er argumentet for Selv synes jeg det var litt rart å ikke bruke
|
Eneste caset det feiler litt på er hvis bruker setter "font-size:" > 100% på html (f.eks browser skalering). Var mest siden de andre maksbreddene vi har satt også er px, så vil være litt merkelig hvis bare Ser ut som de fleste bruker |
@navikt/core/tokens/docs.json
Outdated
| @@ -42,6 +42,7 @@ | |||
| "description": "@media (max-width: 1279px)" | |||
| } | |||
| ], | |||
| "max-width": [{ "name": "--a-max-width-text", "value": "576px" }], | |||
There was a problem hiding this comment.
Vet ikke hva som blir mest riktig, men en annen mulighet er kanskje "text-max-width". Kanskje Sjur har innspill her.
There was a problem hiding this comment.
Det gir mening i forhold til hvordan vi har bygd opp token-navn. Da nevner vi hva som styles først, så hva variasjonene er. Så "text-max-width" funker. Kan dra det enda lenger å sette "max" på slutten, om det kan bli flere bredder: "text-width-max".
Description
Egen token for max-width som tilpasses maksimal leselengde
Change summary
surface-subtlesom option for bakgrunnsfarge. Har nå white, gray-50 og gray-100 tilgjengelig--a-max-width-text: 36remtextsom er--a-max-width-text+padding-inlineDiskusjon
--a-max-width-textmest semantisk riktig her?width="text"ikke endrer seg basert pågutters, når alle de andre breddene gjør dette 🤔TODO: Bør denne endres fra rem -> px?