Legger til mørk bakgrunnsfarge i GridRow #1734
Conversation
|
Usikker på om rekkefølge på commits har noe å si her? Vurderte også å legge inn samme prop på |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1734.westeurope.2.azurestaticapps.net |
| const [bgColor, setBgColor] = useState(backgroundColors[0]); | ||
| const [bgDarkColor, setBgDarkColor] = useState(backgroundDarkColors[0]); |
There was a problem hiding this comment.
Endret forresten default valg i dropdown i eksempelet til Ingen farge, det var før jeg fant ut at eksempelet brukes på https://design.sparebank1.no/komponenter/grid/
Kan endre tilbake hvis dere ønsker
Rekkefølgen har ikke noe å si, det viktigste er at hver pakke har sin egen commit slik at endringer blir publisert på rett sted i npmjs. Hadde helt sikkert vært nyttig med samme prop i |
Beskrivelse
Legger til en
backgroundDarkprop tilGridRowfor å styre bakgrunnsfarge på native darkmode. Jeg la til eksempel kode for denne funksjonaliteten iexamples/grid/GridRow-background.jsxfordi det er et samspill i bakgrunnsfarger på lightmode og darkmode. Ser at dette eksempelet også er brukt på https://design.sparebank1.no/komponenter/grid/ . Bare si ifra hvis funksjonaliteten istedet skal splittes ut i eget eksempel.Motivasjon og kontekst
Man kunne ikke styre bakgrunnsfarge i
GridRownative darkmode før, den ble satt til--ffe-farge-svartsom default. Det er designbehov rundt å kunne sette bakgrunnsfarge til f.eks--ffe-farge-nattpå native darkmode.Testing
Testet manuelt i eksempelet nevnt ovenfor. Laget enhetstester tilsvarende det som fantes fra før. Det er noe css specificity i spill her med at
&--bg-dark-klasser må være definert etter&--bg-for å overskrive bakgrunnsfarge hvis bådebackgroundogbackgroundDarker satt. Skulle gjerne hatt en automatisk test på dette men jest enhetstester er nok ikke riktig sted for det.