Faktisk oppdatert erklering #1264
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oddsor
force-pushed
the
faktisk-oppdatert-erklering
branch
2 times, most recently
from
ebfc6bf to
a454f98
Compare
|
Er det gjort litt for mye i denne PR'en? Kanskje oppdatere erklæringen for seg og så fikse modalene? |
|
Nå er kanskje ikke jeg designer, men er det litt mer sammentrøkket i den oppdaterte modalen?
Og hva har skjedd med stripen mellom periodene? |
eirikv
reviewed
Nov 6, 2024
src/AvtaleOversikt/Taushetserklæring/TaushetserklæringTekst.tsx
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
src/komponenter/alleredeOpprettetTiltak/GodkjennAvtaleMedAlleredeOpprettetTiltak.tsx
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
src/komponenter/alleredeOpprettetTiltak/innholdsvisning/AlleredeOpprettetAvtale.tsx
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
src/komponenter/alleredeOpprettetTiltak/OpprettAvtaleMedAlleredeOpprettetTiltak.tsx
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
src/komponenter/alleredeOpprettetTiltak/innholdsvisning/AlleredeOpprettetAvtaleDetaljer.tsx
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
src/komponenter/alleredeOpprettetTiltak/innholdsvisning/AlleredeOpprettetAvtaleDetaljer.tsx
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
src/komponenter/alleredeOpprettetTiltak/innholdsvisning/alleredeOpprettetAvtale.less
Outdated
Show resolved
Hide resolved
eirikv
reviewed
Nov 6, 2024
sindredl
reviewed
Nov 6, 2024
eirikv
reviewed
Nov 7, 2024
eirikv
reviewed
Nov 7, 2024
eirikv
reviewed
Nov 7, 2024
eirikv
reviewed
Nov 7, 2024
eirikv
reviewed
Nov 7, 2024
sindredl
reviewed
Nov 7, 2024
src/komponenter/alleredeOpprettetTiltak/innholdsvisning/RadInfo.tsx
Outdated
Show resolved
Hide resolved
sindredl
reviewed
Nov 7, 2024
Oddsor
force-pushed
the
faktisk-oppdatert-erklering
branch
2 times, most recently
from
44b81d9 to
1a77364
Compare
Enig, la til spacing på headeren: aa8a625
Designer sier at de skal vekk! |
Oddsor
force-pushed
the
faktisk-oppdatert-erklering
branch
2 times, most recently
from
b260aaf to
e2e926c
Compare
Når man godkjenner på vegne av noen og får opp modalen for å godkjenne avtale på deltaker som allerede har avtaler, vil man ende opp i en evig loop når man avbryter. Dette skyldtes at avbryt-knappen i godkjenn-modalen ble tolket som en submit. Problemet kan fikses ved at man flytter modalen ut av form-elementet.
Hvis arbeidstreningsavtaler er "read only" vil ikke menyen fungere som forventet.
For å unngå flere problemer med at infomodalene ser ulike ut så forsøker vi å lage en felleskomponent for disse modalene.
useRef må flyttes over kodesnutten som returnerer null-verdi hvis modalen ikke er åpen
"open"-prop på modalen skal styre om modalen er åpen eller ikke
Oddsor
force-pushed
the
faktisk-oppdatert-erklering
branch
from
47bad4d to
fa155a2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Oppdatert et par infomodaler:
Taushetserklæring
Før
Etter
Allerede registrert avtale
Før
Etter
Godkjenn allerede registrert avtale
Før
Etter
Bekreftelsesmodal
Før
Etter