Altinn / altinn-design-system

Design system v2
https://designsystem.altinn.studio/
BSD 3-Clause "New" or "Revised" License
10 stars 5 forks source link

Installation: Simplify installation process for consumers #214

Open TomasEng opened 1 year ago

TomasEng commented 1 year ago

Description

Webpack users who install the design system have to modify their settings because of png file imports in the Map component. We want the design system to be easy to install, so this should not be necessary. Can we find a way to make the component work without having to rely on additional configuration by consumers?

Additional Information

No response

DanRJ commented 1 year ago

For dokumentasjonens del: OED er interessert i å ta i bruk dette prosjektet, slik at man kan gjenbruke relevante komponenter. Det blir en gevinst for oss og forhåpentligvis dere hvis vi får mulighet til å delta.

Vi er derimot ikke interessert i å ta i bruk en pakke som krasjer OED og tvinger oss til å installere ekstra unødvendige pakker (file-loader) eller ikke peker mot visse byggmål / standarder. Dette går på PNG-problematikken men også react-leaflet sin lite fleksible byggmål (https://github.com/PaulLeCam/react-leaflet/pull/885#issuecomment-939290018).

Det ser ut som det finnes ulike måter å komme seg rundt den utfordringen, dog har ikke innsikten eller kompetansen til å vite om det fungerer bra.

Tenker dette er nyttig dokumentasjon for de som tar over vedlikehold av designsystemet.

TomasEng commented 1 year ago

Avventer med denne til vi har avklart hvilke komponenter som skal flyttes over til Digdir-designsystemet. Dette arbeidet starter neste uke. Hvis kartkomponenten skal flyttes dit, har det liten hensikt å gjøre et arbeid på å flytte den ut i en egen pakke nå.

DanRJ commented 1 year ago

Kan nevnes nå at OED har modernisert seg og tilpasset seg react-leaflet sine bastante krav. Det funker nå med altinn-design-system. Dog hvis andre prosjekter med tidligere targets enn ES2019 tar inn denne pakken, vil det knekke for de og.