Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bruk av px og em i css #106

Closed
mikaelinmeta opened this issue Feb 12, 2024 · 9 comments
Closed

Bruk av px og em i css #106

mikaelinmeta opened this issue Feb 12, 2024 · 9 comments
Assignees

Comments

@mikaelinmeta
Copy link

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
Copy link
Contributor

Veldig ønsket fra Grunnforurensnings-prosjektet også!

@crolsson
Copy link
Contributor

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

@crolsson
Copy link
Contributor

crolsson commented Jun 4, 2024

Er det gjort noe med dette ønsket?

@aurorascharff
Copy link
Contributor

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

@mikaelinmeta
Copy link
Author

@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
Copy link

IngSchei commented Oct 2, 2024

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

@ohp-inmeta
Copy link
Contributor

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

Ikke på en stund, desverre..

@ohp-inmeta
Copy link
Contributor

ohp-inmeta commented Oct 17, 2024

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

@ohp-inmeta
Copy link
Contributor

Da er dette inkludert i md-css versjon 3.8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants