Meldingsboks og alert message - optisk plassering av ikoner #2602
Replies: 4 comments 2 replies
-
Ja eagle eye Kristine, er enig at Xene er litt for lavt på kompakt komponentene, både message box og system message. Om paddingen til toppen kunne bli 2 px mindre, ser dette da godt ut. For min del er det greit å vise 2x X fordi det er nok visuell forskjell mellom ikon og handling, handlingen ligger på typisk plass hvor man forventer den, og mange løsningene bruker liknende så tror ikke dette er uvanlig. Nysgjerrig på hva andre synes. |
Beta Was this translation helpful? Give feedback.
-
Enig i at lukk-knappen kan justeres litt opp så den kommer mer på linje med de andre. Men er enig med Ivana, synes ErrorMessageBox fungerer slik den er. Jeg kan lage et issue (#2623) på å justere knappeposisjonen :) |
Beta Was this translation helpful? Give feedback.
-
Ser ut som det er noe alignment issues jevnt over i MessageBox, tar en kikk på det. Men trenger vi egentlig ikonet i compact varianten? Ref designprinsipp #3 fra profilen vår
Når det er sagt, så ser skjermbildet ovenfor litt forskjellig ut fra hvordan MessageBox fremstår i biblioteket. Har dere satt inn komponenten direkte og ikke overstyrt innstillingene? |
Beta Was this translation helpful? Give feedback.
-
Tok en kikk i Figma-branchen. Jeg synes x'en fortsatt henger litt langt ned rent optisk, relativt ikonene. I tillegg, ved nærmere inspeksjon så ser ikonene litt for store ut i compact versjonen relativt til hele containeren 🤔 Det må nok ryddes litt i hvordan ikonene er gruppert og spacet med Auto-Layout for å få dette til å sitte bra. Det ser ut til at det har blitt brukt litt kreativ plassering innad i containerene for hvert ikon, og det skaper følgefeilene vi nå ser her. |
Beta Was this translation helpful? Give feedback.
-
Designerne i seOpp jobber med denne komponenten nå og vi syns plasseringen av ikonene og Xen ser skjevt ut - mulig de burde sentreres på midten fremfor i bunn. Ekstra tydelig på mobil siden de kommer nærme hverandre. Også rart med X som ikon på error siden det da blir 2 Xer.
Beta Was this translation helpful? Give feedback.
All reactions