You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Supert at FileUpload er i Beta og at den er anrettet fleksibelt i Figma 💪
Det ble bedt om forslag til justeringer, jeg har et innspill til formspråket.
Ser at det er brukt CTA tertiary icon button til trash og siden CTA buttons har 4px corner radius default i Aksel så blir det mismatch mellom radius verdiene, det vises på trash hover og active state i Items delene, forskjellen blir tydelig da den står i såpass symmetrisk tilknytning til file icon. Siden CTA buttons ikke endres for kun ett tilfelle så hadde det kanskje vært greit å følge opp med samme radius for et mer balansert totalutrykk, enten gjennomgående på hele komponentet som vist her eller på file icon containeren som har 12 px radius nå. Hvis det ble 4px så ville det forøvrig matchet andre Aksel komponenter som Select, Search, Text field, Combobox, Text area, Date Input, CopyButton osv.
The text was updated successfully, but these errors were encountered:
Tilbakemelding
Supert at FileUpload er i Beta og at den er anrettet fleksibelt i Figma 💪
Det ble bedt om forslag til justeringer, jeg har et innspill til formspråket.
Ser at det er brukt CTA tertiary icon button til trash og siden CTA buttons har 4px corner radius default i Aksel så blir det mismatch mellom radius verdiene, det vises på trash hover og active state i Items delene, forskjellen blir tydelig da den står i såpass symmetrisk tilknytning til file icon. Siden CTA buttons ikke endres for kun ett tilfelle så hadde det kanskje vært greit å følge opp med samme radius for et mer balansert totalutrykk, enten gjennomgående på hele komponentet som vist her eller på file icon containeren som har 12 px radius nå. Hvis det ble 4px så ville det forøvrig matchet andre Aksel komponenter som Select, Search, Text field, Combobox, Text area, Date Input, CopyButton osv.
The text was updated successfully, but these errors were encountered: