-
Notifications
You must be signed in to change notification settings - Fork 45
chore(docs): color-scheme behavior #4491
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
base: main
Are you sure you want to change the base?
Conversation
|
|
Preview deployments for this pull request: www - |
|
|
||
| `data-color-scheme` kan bli satt til `light`, `dark` eller `auto`. `auto` vil følgje systeminnstillingane til brukaren. | ||
|
|
||
| `color` og `background-color` blir redeklarert til standard verdiar på elementet med `data-color-scheme`. Dette er nødvendig fordi variablane ikkje blir arva forbi det punktet dei blir redefinert. |
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.
"ette er nødvendig fordi variablane ikkje blir arva forbi det punktet dei blir redefinert."
Det forsto jeg ikke helt 😇 Er litt av poenget også å sette background/color så man sikrer at kontrast er tilstrekkelig ivaretatt? dersom man legger en komponent som setter color men ikke background inni noe med data-color-scheme så må vi vite at background på elementet som satt data-color-scheme ble rett?
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.
Ja det kan kanskje formuleres bedre. problemet (fra #4478) er eksemplifisert her, der jeg har kommentert ut [data-color-scheme] fra selektoren. data-color-scheme="dark" i den første boksen der verken color eller background er definert: background er hvit fordi det er user-agent default, og tekst er color er --ds-color-neutral-text-default: #292c30; fordi color arvet variabelen før den ble redefinert av color-scheme.
Så vi må redeklarere både color og background-color for å sikre kontrast der vi setter data-color-scheme.
Kan prøve å reformulere det i morgen!
| } | ||
| ``` | ||
|
|
||
| You can override this by setting custom values for `color` and `background-color` on the element. |
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.
| You can override this by setting custom values for `color` and `background-color` on the element. | |
| You can override this by setting custom values for `color` and `background-color` on the element, or globally by matching our selector. |
| } | ||
| ``` | ||
|
|
||
| Du kan overstyra dette med å setje eigne verdiar for `color` og `background-color` på elementet. |
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.
| Du kan overstyra dette med å setje eigne verdiar for `color` og `background-color` på elementet. | |
| Du kan overstyra dette med å sette eigne verdiar for `color` og `background-color` på elementet, eller globalt ved å bruke samme selector som oss. |
resolves #4478
er det korrekt nynorsk?!