Chore: Bytt ut bruk av familie-form-elements med komponenter fra Aksel #1639
Conversation
…eriode skjema+ fiks styling
There was a problem hiding this comment.
Lesevisning
Ikke lesevisning
| @@ -30,7 +30,6 @@ | |||
| "@navikt/ds-tokens": "^5.12.0", | |||
| "@navikt/familie-backend": "^10.0.7", | |||
| "@navikt/familie-datovelger": "0.2.0", | |||
| "@navikt/familie-form-elements": "^14.0.0", | |||
...nd/komponenter/Fagsak/Foreldelse/ForeldelsePeriode/FeilutbetalingForeldelsePeriodeSkjema.tsx
Outdated
Show resolved
Hide resolved
| ? foreldelsevurderinger[periode.foreldelsesvurderingstype] | ||
| : '' | ||
| } | ||
| value={skjema.felter.foreldelsesvurderingstype.verdi} |
There was a problem hiding this comment.
Kan ikke mappe om til streng lenger når det er lesevisning, fordi ved lesevisning rendrer vi nå fortsatt selve radiogruppen og ikke en tekststreng. Dette feltet blir satt til periode.foreldelsesvurderingstype i en useEffect lenger opp her, så det burde fortsatt ha riktig verdi i lesevisning
There was a problem hiding this comment.
Se bilder i kommentar her #1639 (comment)
| > | ||
| {behandlingsTyper.map(opt => ( | ||
| {Object.values(Behandlingstype).map(opt => ( |
There was a problem hiding this comment.
behandlingsTyper var en liste med de to verdiene som finnes for Behandlingstype så bruker heller enumen direkte
| erLesevisning={erLesevisning} | ||
| tekstLesevisning={underavsnitt.fritekst || 'Fritekst ikke utfylt'} | ||
| className={ | ||
| erLesevisning && !underavsnitt.fritekst ? 'lesevisning_ikke_utfylt' : '' |
There was a problem hiding this comment.
Fant ingen styling trigget av lesevisning_ikke_utfylt
| onChange={event => onChangeVergeType(event)} | ||
| > | ||
| <option disabled={true} value={''}> | ||
| Velg vergetype | ||
| </option> | ||
| {vergeTyper | ||
| {Object.values(Vergetype) |
There was a problem hiding this comment.
vergeTyper var en liste med alle verdiene i Vergetype-enum, med unntak av UDEFINERT, som uansett blir filtrert ut her
...er/Fagsak/Vilkårsvurdering/VilkårsvurderingPeriode/Aktsomhetsvurdering/GradForsettSkjema.tsx
Outdated
Show resolved
Hide resolved
…nter med samme tekst)
src/frontend/komponenter/Fagsak/Fakta/FaktaPeriode/FeilutbetalingFaktaPeriodeSkjema.tsx
Show resolved
Hide resolved
src/frontend/komponenter/Fagsak/Høyremeny/Totrinnskontroll/Totrinnskontroll.tsx
Show resolved
Hide resolved
...gsak/Vilkårsvurdering/VilkårsvurderingPeriode/Aktsomhetsvurdering/ReduksjonAvBeløpSkjema.tsx
Outdated
Show resolved
Hide resolved
Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-17446
Skriver oss bort fra
familie-form-elementsog bruker heller komponenter direkte fra Aksel.Burde nok leses commit for commit.Tror alt i ett kan være lettere, har blitt litt mye frem og tilbake på slutten.Har lagt ved en del skjermbilder, men ikke over alt fordi jeg mener det ikke er nødvendig.
Største forskjellen er nok at flere elementer nå bruker Aksel sin readonly-prop, som gjør at man rendrer komponenten nesten likt uansett om du kan redigere den eller ikke. Se eksempel for forskjell her.
Før
Etter
