-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Legge til utvidet støtte for dark mode i card select #765
Conversation
|
cc @perweum ser dette greit ut for deg? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kjempefint lite PR <3
Tror det heter color
og ikke textColor
i CSS, men kanskje begge fungerer i Chakra. Uansett så foretrekker vi color
.
Co-authored-by: Kristofer Giltvedt Selbekk <[email protected]>
Her har det skjedd en liten glipp. Jeg hadde en prat med Live og vi går heller vekk fra å bruke bakgrunnsfarge på outlined komponenter. Der en komponent bruker bakgrunnsfarge i en state (som ghost-active) skal det være definerte egne farger for dark og light-mode. Jeg skal prøve å få samlet dokumentasjonen i kveld sånn at du kan justere komponentene. |
Her er link til en visualisering av dark og lightmode for å vise forskjellene. Det at vi har fjernet bakgrunnen på flere komponenter er ikke implementert enda i kode, men vi fant ut at det er enkleste måte å håndtere ulike bakgrunnsfarger. I digital bruker vi "Night" som bg i darkmode. Vi holder fortsatt på å sette designet i darkmode for alle komponenter så det er veldig fint med feedback sånn som dette. |
@selbekk, Hanne hadde en bra tilbakemelding på designet på darkmode, ved at det ikke vil fungere der vi må bruke komponenten som overlay (f.eks. ved kart). Jeg lurer på om det mulig å legge til en ny prop med en solid bakgrunn, hvor man kan definere bakgrunn selv? Så kan IT bruke "Dark Grey" og vi kan bruke "Night", men at komponenten fortsatt har en "White Alpha 100" bakgrunn som default. |
Bakgrunn
Per dags dato er cardet i Card Select alltid hvitt, uavhengig av om mode er light eller dark. (Se bildet) Dersom dette skal endres slik at bakgrunnsfargen støtter dark mode, må også tekstfarge oppdateres for å synes på ny dynamisk bakgrunn.
Løsning
Jeg har lagt til dynamisk bakgrunnsfarge og da også tekstfarge for card i cardselect. Fant ingen skisser i Figma for darkmode i Card Select enda, så jeg har tatt utgangspunkt i bakgrunnsfargene for fargevalget.
Det ser da slik ut:
Jeg har prøvd å bruke samme tilnærming som eksisterende komponenter med støtte for darkMode, men si gjerne ifra dersom det er ønskelig å gjøre noe annerledes 😊