SpareBank1 / designsystem

SpareBank 1's design system and component library.
https://design.sparebank1.no
MIT License
100 stars 87 forks source link

Kunne DetailsCard hatt bakgrunn @ffe-grey-charcoal-darkmode i stedet for @ffe-farge-natt på darkmode? #2137

Closed MariaOverlierBerg closed 1 day ago

MariaOverlierBerg commented 2 days ago

På betalingssidene våre bruker vi @ffe-grey-charcoal-darkmode i lister og cards og diverse, og ser at blant annet oversikt også gjør dette. Hadde det vært mulig å brukt den på DetailListCard-komponenten i @sb1/ffe-lists-react? Det blir hvert fall litt mindre konsekvent på sidene våre når vi har tatt den i bruk.

Skjermbilde 2024-07-01 kl  10 46 53Skjermbilde 2024-07-01 kl  10 42 48Skjermbilde 2024-07-01 kl  10 42 59Skjermbilde 2024-07-01 kl  10 43 10

pethel commented 2 days ago

Takk før att du spør. Den blå versionen er vad som ligger i figma så dette må vell bli ett spørsmål før design da.

MariaOverlierBerg commented 2 days ago

Burde jeg legge dette inn i ønskelista i figma i stedet? 😊

antidecaf commented 1 day ago

@ffe-grey-charcoal(-darkmode) hører til den gamle fargepaletten og bør helst ikke brukes. Fargen er egentlig kun fortsatt tilgjengelig i og med at den er deprecated og ikke fullt ut slettet ennå. Det samme gjelder alle farger som ikke starter på @ffe-farge i colors.less. Så velg gjerne en av fargene fra den nåværende paletten, om noe. :)

MariaOverlierBerg commented 1 day ago

Aha! Skjønner! 😄 Da går vi nok for @ffe-farge-koksgraa som ligner mest. Vi brukte farge-natt før, men etter prat med desginer, synes vi det ser best ut med en lysere svart / mørkere grå enn mørkeblå på darkmode. 😊

HeleneKassandra commented 1 day ago

Jeg legger til grey-charcoal som ett farge alternativ i detailListCard i darkmode, den er som Anders sier en gammel farge og bør egentlig ikke brukes, men siden vi mangler retningslinjer og alternativer i darkmode. Så legger jeg den til uansett, sånn at vi får hvertfall byttet over til denne komponenten, også får vi heller fjerne fargen senere når man har retningslinjer