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

Rettingar etter testing av funksjonalitet og UU #551

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ingfo
Copy link
Contributor

@ingfo ingfo commented Jun 6, 2024

For detaljar sjå commit-meldingar

@ingfo ingfo changed the title Rettingar etter testing Rettingar etter testing av funksjonalitet og UU Jun 6, 2024
@ingfo
Copy link
Contributor Author

ingfo commented Jun 6, 2024

@ingfo ingfo marked this pull request as ready for review June 6, 2024 14:08
@ingfo ingfo requested a review from a team as a code owner June 6, 2024 14:08
i migreringsmodal

slik at brukaren sparer eit klikk og oppførselen samsvarer betre med knappeteksten.
Slik at brukarane ikkje mister informasjonen i arbeidslista si fordi migreringa feila på vår side.
slik at den vert påverka av innstillingane brukaren har gjort i nettlesaren sin.

Brukar det også på stader der ting risikerer skjule innhald om det ikkje skalerer saman med teksten.
Byttar ut 24(px) med 1em etter mønster frå designsystemet og Huskelappikon_bakgrunnsfarge.svg
@@ -14,7 +14,7 @@
<style>
#internflate {
width: 100%;
height: 75px;
height: calc(75/16 * 1rem);
Copy link
Contributor

Choose a reason for hiding this comment

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

Den her kan vi endre til å matche dagens høyde på dekoratøren

Suggested change
height: calc(75/16 * 1rem);
height: 4.125rem;

@ingfo
Copy link
Contributor Author

ingfo commented Aug 7, 2024

I staden for å prøve å flette inn nokre månadar med andre endringar skal eg gjere dette på nytt.
Eg let PR-en liggje open så eg hugsar det.

@ingfo ingfo marked this pull request as draft August 7, 2024 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants