Chat: fjern border, gjør svg mindre, bytt om "subtle" og "neutral" #2077
Conversation
🦋 Changeset detectedLatest commit: d6c714b The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
| @@ -41,16 +40,16 @@ | |||
|
|
|||
| .navds-chat__avatar svg { | |||
| align-self: center; | |||
| height: 100%; | |||
There was a problem hiding this comment.
Skjer noe alignment bugs her når denne fjernes, usikker hvorfor 🤔
Etter:
Før:
There was a problem hiding this comment.
Chrome og avrunding 🙄 Eventuell fiks blir hacky, tror jeg... Skal undersøke.
There was a problem hiding this comment.
Ble ikke så ille som jeg fryktet 😅
There was a problem hiding this comment.
LGTM 🚀
En ting jeg så først nå var at variant "subtle" og "neutral" er reversert. Egentlig skal Neutral være den "mørkeste" varianten, mens subtle er den lyseste. Må evt fikses i CSS + endre default variant fra subtle til neutral. Dette er i teorien en breaking change, men kan kanskje gjøre noen github-søk å se om noen har oppdatert til 4.2.0 + bruker Chat. Hvis det brekker noen apper trenger vi ikke å fikse det nå
HalvorHaugan
changed the title
Fjernet border og satte maks-størrelse på svg til 24x24px: #2075
Linda gir beskjed om endringen til de som bruker komponenten.
Usikker på om det er patch eller minor...?