fix: replace classnames with clsx

[zenabii]

• Alle komponent pakker har fått classnames pakken avinstallert
• clsx er installert i root, her må vi teste litt hva som fungerer og ikke. Fin overgang til reisen å samle alle komponenter til én pakke
• Rettet opp noen skriveleifer her og der
• classnames pakken er på ca 1,2k kb, clsx er på 239 kb
• cn og cx erstattet med clsx i koden i alle komponenter

Denne PR'en burde vi ta en felles gjennomgang på og idémyldre litt hvordan vi vil angripe oppgaven å gå fra mange pakker til én komponentpakke og proper tree shaking :)

🎯 Sjekkliste

• [ ] Nye features er dokumentert (sjekk Contributing om du er usikker på hva som trengs av dokumentasjon)
• [x] Testet responsivitet og UU (tastaturnavigasjon, skjermleser)
• [x] pnpm build og pnpm ci:test gir ingen feil

[zenabii]

Noe har skjedd i pnpm lock filen som påvirker import i select.tsx - noen som kunne tenkt seg å sparre med meg på dette? 😄

[ivarni]

Noe har skjedd i pnpm lock filen som påvirker import i select.tsx - noen som kunne tenkt seg å sparre med meg på dette?

Prøv pnpm install på rota til prosjektet.

[fremtind-bot]

✅ Forhåndsvisning: https://jokul.fremtind.no/preview/fix--replace-classnames-with-clsx/ 🔍 Commit: 55a2ee081922b2aacd0b3aee28116b8da1aeb7b9

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

[ivarni]

Jeg testet hvordan det blir etter publish dersom clsx avhengigheten bare er definert i root-pakka. Brukerne må da manuelt installere den pakka siden det ikke blir med noe informasjon om at den trengs ut på repoet.

Dette kunne vært løst veldig elegant med pnpm sin catalog protokoll meeen dessverre støtter ikke lerna dette enda og lerna-lite har ingen umiddelbare planer. Jeg syns det er mye verdi i å sikre at vi bruker samme versjoner av avhengigheter på tvers av pakkene våre men tror at vi da må inn med syncpack eller noe lignende.