Conversation
johancarnegard
requested review from
MCFK,
NicklasWei,
OlofFredriksson,
TMFKse,
dvitamin,
ext and
mitra-Re
as code owners
|
|
|
||
| // BACKGROUND - ACTION | ||
|
|
||
| --fkds-color-action-background-secondary-hover: #{$palette-color-bluebell-15}; // global |
There was a problem hiding this comment.
Ser att vi gjort så på alla nu men // globla kommentaren, fyller den något syfte? Inte tekniskt iaf.
| border-color: var(--f-border-color-chip-active); | ||
| color: var(--f-text-color-chip-active); | ||
| background: $chip-color-background-checked; | ||
| border-color: transparent; |
There was a problem hiding this comment.
Ska denna alltid i 100% alla lägen varje gång vara transparent eller är det ett val vi gjort för vårat tema? Eller borde vi ha en $chip-color-border-checked som är transparent (behöver inte vara en semantisk variabel).
Tänker att det blir symetriskt och bra då vi har $chip-color-text-checked och $chip-color-background-checked.
There was a problem hiding this comment.
Det skulle väl i och för sig kunna vara något som man skulle vilja styra via temat, men i så fall borde vi göra samma sak för primär knapp.
johancarnegard
force-pushed
the
feat/semantic-token-chip
branch
from
86b3ae3 to
d0f1d3e
Compare
johancarnegard
force-pushed
the
feat/semantic-token-chip
branch
from
d0f1d3e to
f53f345
Compare
|
🎉 This PR is included in version 5.42.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
4 participants
No description provided.