Konsolidere spacingskala

[onordbo]

Discussed in #2375

^{Originally posted by onordbo October 12, 2021}
Etter litt debatt på Forum rundt bruken og dokumentasjonen av spacingverdiene våre, ble vi enige om at det ikke var noe særlig nytteverdi for oss å skille mellom komponent- og layoutverdier. Per i dag bruker vi t-skjorte størrelser (xxs, xs, s, m, osv osv). Slik jeg har forstått det på den faktiske implementasjonen i Jøkul så varierer dette mellom forkortelser av størrelsen og hele ordet. Såvidt jeg husker så hadde @Saegrov noen meninger om hvordan vi burde navngi selve klassene? Fint om vi deler litt innsikt og eventuelle andre meninger her. Jeg kan fyre opp et issue og ta meg av Figma-dokumentasjonen når vi er i boks.

Vi slår nå sammen spacing og går vekk fra to forskjellige kategorier.

To-do's

• Definere navngivning for skalaen
• Oppdatere Figma med de nye verdiene og klassenavnene
• Oppdatere koden og portalen med nye verdier (Under arbeid, feat: new spacing scale #2381.)

[onordbo]

Jøkul Bibliotek 2.0 oppdatert med ny skala.

cc @Murstam

[Murstam]

Burde staten ha t-skjortenavn i Figma slik at designere bruker samme benevnelse og det blir tydeligere for utviklere i inspect? @onordbo

[onordbo]

Jeg er ikke helt sikker på at jeg henger med her, @Murstam. Tenker du på selve "klossen" som viser avstanden visuelt da? At den burde hatt en merkelapp? Eller tenker du på merkelappene til variantene?

[onordbo]

Figma biblioteket er nå ferdigstilt og denne saken er klar fra designsiden, og vil automatisk lukkes sammen med merging av #2381.

[piofinn]

Stemmer for at vi fjerner "Variant" i Figma (siden den ikke lenger varierer) og kaller "state" for "Størrelse" e.l. Vi må også fikse dette før jeg blir gal:

😬😬😬

Tror man kan endre rekkefølge på valgene hvis man markerer komponenten i biblioteket (man kan dra tag'ene i Variants-panelet i hvert fall). Tagger deg for dette, jeg @onordbo 😁

Edit: Fiksa det sjæl i en Figma-branch 😅 Bare å dra inn den som heter Spacing-tweaks hvis dere er enige med meg!

[onordbo]

Wups, den lista glapp jeg @piofinn, bra catch! Merget ✅