search

miljodir / md-components

Design-komponenter for Miljødirektoratet (CSS og React)
https://miljodir.github.io/md-components
MIT License
3 stars 3 forks source link

Bruk av px og em i css #106

Closed mikaelinmeta closed 1 month ago

mikaelinmeta commented 9 months ago

Vi i Tilde-prosjektet har til hensikt å gjenbruke CSS-pakken som er tilgjengelig i dette repoet for å holde enhetligheten så mye som mulig med designsystemet.

Men når vi ser på CSS-en, har vi kommet over 2 ting. Bruken av px og em. Vi vil foreslå å erstatte begge disse med rem. Px til rem for responsivitet og accessibility. Er bruken av "em" et bevisst valg? Jeg er ikke så kjent med det, og det føles litt DX-uvennlig at du må holde styr på parents font-size.

Vi har ingen problemer med å opprette en pull request med disse endringene hvis ønskelig. Og hvis dere er enig, selvfølgelig 🙂

kajsaeggum commented 9 months ago

Veldig ønsket fra Grunnforurensnings-prosjektet også!

crolsson commented 7 months ago

Høres fornuftig ut. Lag en PR om det ikke allerede er utført :)

crolsson commented 6 months ago

Er det gjort noe med dette ønsket?

aurorascharff commented 2 months ago

@mikaelinmeta Det virket som alle er enige, har dere noen planer om å gjøre det?

mikaelinmeta commented 2 months ago

@aurorascharff Vi er ikke helt i tråd med CSS-en siden vårt design ble utformet før designsystemet ble opprettet, så vi har noe overstyrende CSS og ligger muligens et par versjoner bak (vanskelig å si når React og CSS-pakkingen er i samme repo).

Men jeg antar at Ingrid/noen ansvarlig kan ta en avgjørelse på hvem og når ressurser trengs for denne endringen, ikke sant?

Btw, her er en nice extension i VSCode for px -> rem

IngSchei commented 2 months ago

@ohp-inmeta har du mulighet til å se på denne?

ohp-inmeta commented 2 months ago

@ohp-inmeta har du mulighet til å se på denne?

Ikke på en stund, desverre..

ohp-inmeta commented 1 month ago

PR for dette klar: #187 @mikaelinmeta siden du meldte dette inn, kanskje du også vil ta en kikk?

ohp-inmeta commented 1 month ago

Da er dette inkludert i md-css versjon 3.8.0