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

Develop fjern colored bg #2291

Merged
merged 4 commits into from
Sep 16, 2024
Merged

Develop fjern colored bg #2291

merged 4 commits into from
Sep 16, 2024

Conversation

pethel
Copy link
Contributor

@pethel pethel commented Sep 12, 2024

Fjerner onColoredBg .. tror vi bare må lage en version av den der det er støtett fordi props på sendes osv. Fikk også fonterna att funka med inline

@pethel pethel marked this pull request as ready for review September 12, 2024 08:08
@pethel pethel requested a review from a team as a code owner September 12, 2024 08:08
@pethel pethel force-pushed the develop_fjern-colored-bg branch from dc2399c to 413c5b3 Compare September 12, 2024 08:10
Copy link
Contributor

@antidecaf antidecaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fontene bør vel helst ikke slettes?

@pethel
Copy link
Contributor Author

pethel commented Sep 12, 2024

Fontene bør vel helst ikke slettes?

Dom er inline nu istellet. Det så ut att funka. Vi trenger ikek ha dom inline kanskje? Cache og sånt?

@pethel pethel force-pushed the develop_fjern-colored-bg branch from 413c5b3 to 74b09c1 Compare September 12, 2024 08:53
@antidecaf
Copy link
Contributor

Dom er inline nu istellet. Det så ut att funka. Vi trenger ikek ha dom inline kanskje? Cache og sånt?

Men de kan vel fortsatt ikke slettes fra ffe-webfonts? Da brekker jo sb1-fonts.less helt?

@pethel
Copy link
Contributor Author

pethel commented Sep 12, 2024

Dom er inline nu istellet. Det så ut att funka. Vi trenger ikek ha dom inline kanskje? Cache og sånt?

Men de kan vel fortsatt ikke slettes fra ffe-webfonts? Da brekker jo sb1-fonts.less helt?

Men den filen har vi vell inget forhold til lenger?

@pethel pethel requested a review from antidecaf September 12, 2024 09:29
@antidecaf
Copy link
Contributor

Men den filen har vi vell inget forhold til lenger?

ffe-webfonts er fortsatt en pakke som er tilgjengelig via npmjs og som lastes ned mellom 200 og 300 ganger i uka derfra og vi vet ikke hvordan den brukes overalt. I beste fall er slettingen en breaking change som burde markeres som det i en egen commit, men vi kan i alle fall ikke slette fonts-mappa uten videre.

@pethel
Copy link
Contributor Author

pethel commented Sep 12, 2024

Men den filen har vi vell inget forhold til lenger?

ffe-webfonts er fortsatt en pakke som er tilgjengelig via npmjs og som lastes ned mellom 200 og 300 ganger i uka derfra og vi vet ikke hvordan den brukes overalt. I beste fall er slettingen en breaking change som burde markeres som det i en egen commit, men vi kan i alle fall ikke slette fonts-mappa uten videre.

Hahah nu ser jag... jag slettet feil filer. mente rog slette dom i public/fonts som vi
brukte

@pethel pethel force-pushed the develop_fjern-colored-bg branch from 74b09c1 to 0cdb99d Compare September 12, 2024 09:53
@pethel
Copy link
Contributor Author

pethel commented Sep 12, 2024

Men den filen har vi vell inget forhold til lenger?

ffe-webfonts er fortsatt en pakke som er tilgjengelig via npmjs og som lastes ned mellom 200 og 300 ganger i uka derfra og vi vet ikke hvordan den brukes overalt. I beste fall er slettingen en breaking change som burde markeres som det i en egen commit, men vi kan i alle fall ikke slette fonts-mappa uten videre.

Sånn! Nå har jag sletet rikig sted!

@pethel pethel force-pushed the develop_fjern-colored-bg branch from 12d0268 to d5af780 Compare September 12, 2024 10:26
@antidecaf
Copy link
Contributor

antidecaf commented Sep 12, 2024

Sånn! Nå har jag sletet rikig sted!

Topp! Da gjenstår det bare å finne ut hvorfor lintingen klager på postcss-less i byggene på de siste PR-ene. 😶

Edit:
Lokalt får jeg feilmelding om at ffe-core, ffe-symbols og ffe-webfonts mangler [email protected]

@sb1/ffe-core: Need to install the following packages:
@sb1/ffe-core: [email protected]

@pethel pethel force-pushed the develop_fjern-colored-bg branch from d5af780 to 043a048 Compare September 16, 2024 08:20
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2291.westeurope.2.azurestaticapps.net

@pethel pethel merged commit 88957c0 into develop Sep 16, 2024
3 checks passed
@pethel pethel deleted the develop_fjern-colored-bg branch September 16, 2024 11:02
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

Successfully merging this pull request may close these issues.

2 participants