nve-tooltip har feil font, hjørneradius og farge

NVE / Designsystem

MIT License

1 stars 0 forks source link

nve-tooltip har feil font, hjørneradius og farge #219

Closed gruble closed 2 weeks ago

gruble commented 3 weeks ago

Vi må bruke nve-tokens for disse. @knutnve kan du spesifisere mer?

knutnve commented 3 weeks ago

Enkel jobb:

Tooltip skal ha revered colors. Som vil si fargene flippes, så bg= fg.

Endre bg farge til: neutrals-foreground-primary
Endre fg farge til: neutrals-background-primary
border radius-small
spacing-xsmallpadding sidene og top/bunn.

Angående oppførsel, er det sånn SL komponentet kun er med pil under i midten?

Figma fil https://www.figma.com/design/0eXhyUrUF7fWi1VaphfpEu/04---%E2%9D%96-Komponenter?node-id=2056-115952&t=CpmRWS41aTJJHGgH-4

knutnve commented 3 weeks ago

Skal vi tilby feedback fargene til tooltipen?

tomapedersen commented 3 weeks ago

Kan gjøre det. Dersom jeg skal endre fargene er det ikke en stor sak å legge til variant også

Plassering for pil på tooltip er avhengig av hvor tooltip plasseres og hvor target er. Man kan også sette placement for tooltip for å overstyre, se første eksempel på https://shoelace.style/components/tooltip

tomapedersen commented 3 weeks ago

🛈 : Siden vi nå bruker feedback-fargene, så gjelder ikke punktet med neutrals-foreground-primary / neutrals-background-primary. Istedenfor bruker vi variablene av typen feedback-background-emphasized-neutral