NVE / Designsystem

MIT License
1 stars 0 forks source link

PR: bytte mellom tema variablene #187

Closed amish1188 closed 2 months ago

amish1188 commented 3 months ago

La til muligheten til å bytte mellom 'nve' og 'varsom' temaer. Jeg valgte å bruke tilnærmingen med å lage separate CSS-filer for doc-siden med de respektive tema-tokens, da dette viste seg å være enklere og mer effektivt.

Grunnet begrensninger i JavaScript, hvor importerte filer eller moduler ikke kan fjernes, ville det ha vært problematisk å overskrive 'nve_theme' ved å senere importere 'varsom_theme'. Så selve import av varsom_theme ville gjøre ingenting. En potensiell løsning kunne ha vært å refreshe siden, men dette virket unødvendig og ikke brukervennlig.

Ved å tillate bytte av CSS-tema ved bruk av CSS-theming på root (root.nve eller root.varsom), kan man smootlhy bytte mellom 'nve' og 'varsom' CSS-klasser i DOM.

Jeg har også lagt til et script som automatisk oppdaterer 'nve_theme.css' og 'varsom_theme.css' filene når 'tokenbuild' kjøres - så at vi ikke må oppdatere disse filene manuelt.

I tillegg har jeg lagt til støtte for '.dark' variabler når man bytter til dark mode.

github-actions[bot] commented 3 months ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-187.westeurope.5.azurestaticapps.net

github-actions[bot] commented 3 months ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-187.westeurope.5.azurestaticapps.net

github-actions[bot] commented 2 months ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-187.westeurope.5.azurestaticapps.net

github-actions[bot] commented 2 months ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-187.westeurope.5.azurestaticapps.net