fix: removes scss warnings about mixins

fremtind / jokul

Jøkul er et designsystem utviklet av Fremtind.

https://jokul.fremtind.no

MIT License

65 stars 31 forks source link

fix: removes scss warnings about mixins #4296

Closed ivarni closed 1 week ago

ivarni commented 1 week ago

🎯 Sjekkliste

[x] pnpm build og pnpm ci:test gir ingen feil

ivarni commented 1 week ago

SASS er fortsatt sint på _reset.scss og _theme.scss uten at jeg helt forstå hvorfor

On that note, hvorfor har vi så mange resets?

fremtind-bot commented 1 week ago

Forhåndsvisning: https://jokul.fremtind.no/preview/scss-warnings/ Commit: 5238a96fbc21033554a6f3c3242aab126200b76d

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

piofinn commented 1 week ago

On that note, hvorfor har vi så mange resets?

Det er den fila reset-outline-mixinen bor i, så den blir nok importert mange steder. Edit: Jeg misforsto litt hva du mente. Det ser ut som at både fila med reset-outline og fila med faktisk CSS-reset heter _reset.scss. Og så er det jo en versjon av hver i den gamle core-pakka og en i den nye 😅

piofinn commented 1 week ago

@ivarni Jeg er like forvirret som deg over hvorfor den klager over reset-outline, men følgende endring ser ut til å gjøre susen:

Lurer på om det har noe å gjøre med hvordan media queries håndteres.

ivarni commented 1 week ago

Tar det med i denne jeg