digdir / designsystemet

Designsystemet
https://designsystemet.no
MIT License
68 stars 34 forks source link

ToggleGroup: Design component in Figma #804

Closed mrosvik closed 9 months ago

mrosvik commented 9 months ago

Image

Figma: https://www.figma.com/file/vpM9dqqQPHqU6ogfKp5tlr/Felles-komponenter?type=design&node-id=19969%3A12845&mode=design&t=u8EX4HUZe4EcVg2c-1

mrosvik commented 9 months ago

@Febakke Vil du teste i Figma? :)

Febakke commented 9 months ago

Litt rart at man kan styre størrelse på hver enkelt knapp. Men dette får vi ikke gjort noe med før Figma kommer med støtte for å skjule nøstede egenskaper. 😔

Overstyring av tekst fungerer ikke mellom small og medium/large. Kan være pga medium og large er bygd opp med small som instans inne i komponenten.

Image

Dette gjør også at nøstede egenskaper forsvinner på medium og large. Jeg prøvde å legge til de nøstede egenskapene på de men da får man med dobbelt av size 🤔

Image

Kanskje vurdere å teste å bygge de som like komponenter for å se om de problemene forsvinner.

Trenger vi noen retningslinjer for hvor store knappene skal være? Om du legger inn egen tekst må du stille på de selv for å sette riktig størrelse.

mrosvik commented 9 months ago

@Febakke Ser ut som mange trenger dette ja: https://forum.figma.com/t/expose-nested-instances-at-property-level/30052

mrosvik commented 9 months ago

@Febakke Tekst skal fungere nå mellom bytte av størrelse.

https://github.com/digdir/designsystem/assets/1464915/f1fe4578-b94b-4321-9e24-81df82db4169