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.

Sign up for GitHub

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

Chore: Bytt ut bruk av familie-form-elements med komponenter fra Aksel #1639

Merged
merged 32 commits into from
Feb 21, 2024
Merged

Chore: Bytt ut bruk av familie-form-elements med komponenter fra Aksel #1639

Show file tree
Hide file tree
Changes from 29 commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
9de0b72
Bytt ut Grid og Cell som er deprecated med HStack
idaame Feb 16, 2024
beaca24
Bytt ut FamilieSelect med Select fra Aksel for feilutbetaling fakta p…
idaame Feb 16, 2024
7e630cf
FamilieInput -> TextField
idaame Feb 16, 2024
179a9db
FamilieSelect -> Select for Opprett Behandling modal
idaame Feb 16, 2024
fd40d08
FamilieSelect -> Select for Verge-type
idaame Feb 16, 2024
7b72830
FamilieSelect -> Select for andel som skal tilbakekreves
idaame Feb 16, 2024
81ef990
Erstatt FamilieTilbakeTextArea.tsx med Textarea fra Aksel
idaame Feb 16, 2024
5eb9c1d
FamilieRadioGruppe -> RadioGroup fra Aksel
idaame Feb 16, 2024
c496cea
FamilieRadioGruppe -> RadioGroup for den horisontale radiogruppen
idaame Feb 16, 2024
0668f18
FamilieCheckbox -> Checkbox fra Aksel + slett ubrukt styling
idaame Feb 16, 2024
168de4a
Slett familie-form-elements som ikke brukes lenger
idaame Feb 16, 2024
9468f2a
Lag lesevisningsversjon så radiogruppe ikke rendres når den er i read…
idaame Feb 16, 2024
9736177
ikke map om options, men bruk verdi fra feltet direkte for grovt uaks…
idaame Feb 16, 2024
ab96a91
ikke vis radiogruppe hvis man ikke kan endre den
idaame Feb 16, 2024
304c0d7
kun vis tekst hvis lesevisning for hendelsesårsak + underårsak
idaame Feb 16, 2024
29c332e
slett linje i test som ikke gir mening lenger når vi har textarea med…
idaame Feb 16, 2024
8c7eed6
ikke bruk readonly fra aksel på radio buttons for totrinnskontroll
idaame Feb 16, 2024
5776cc8
Oppdater test nå som vi viser inputfelter i lesevisning
idaame Feb 16, 2024
a15fec5
Oppdater test nå som vi viser inputfelter i lesevisning
idaame Feb 16, 2024
ea4c65a
Oppdater skjult legend for totrinnskontroll for å ikke gå i beina på …
idaame Feb 16, 2024
f22e0dd
Alltid la verdien til radiogruppen være lik skjemaverdien
idaame Feb 16, 2024
fc5a239
Sett verdi på radiogrup for foreldelsesskjema riktig også for lesevis…
idaame Feb 19, 2024
edda9d5
Sett riktig verdi på radiogrup ved lesevisning for aktsomhetsvurdering
idaame Feb 19, 2024
27b73b2
Bruk riktig skjema-felt for å vise lesevisning-verdi
idaame Feb 19, 2024
8f62e5f
Bruk skjema-feltet sin verdi også i lesevisning for GradUaktsomhetSkj…
idaame Feb 19, 2024
efae649
Bruk skjema-feltet sin verdi også i lesevisning for ReduksjonAvBeløpS…
idaame Feb 19, 2024
de38966
Ikke vis label for ReduksjonAvBeløpSkjema.tsx
idaame Feb 19, 2024
cdfe342
Rename label på select så test er happy (skal ikke finnes flere eleme…
idaame Feb 19, 2024
78aaaf8
reverter endringer
idaame Feb 19, 2024
a069e7b
trekk radio group for å tillegge renter ut i egen komponent
idaame Feb 21, 2024
7bdcbc0
forenkle hva man sender inn som prop for å styre om man viser feilmel…
idaame Feb 21, 2024
8e2b927
skjemafelt -> felt
idaame Feb 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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",
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

🙌

"@navikt/familie-http": "^6.0.2",
"@navikt/familie-logging": "^6.0.0",
"@navikt/familie-skjema": "^7.0.6",
Expand Down
8 changes: 0 additions & 8 deletions src/frontend/kodeverk/verge.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,3 @@ export const vergetyper: Record<Vergetype, string> = {
ANNEN_FULLMEKTIG: 'Annen fullmektig',
UDEFINERT: 'Udefinert',
};

export const vergeTyper = [
Vergetype.VERGE_FOR_BARN,
Vergetype.VERGE_FOR_FORELDRELØST_BARN,
Vergetype.VERGE_FOR_VOKSEN,
Vergetype.ADVOKAT,
Vergetype.ANNEN_FULLMEKTIG,
];
halvorbmundal marked this conversation as resolved.
Show resolved Hide resolved
15 changes: 7 additions & 8 deletions src/frontend/komponenter/Fagsak/Brevmottaker/BrevmottakerSkjema.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,8 @@ import React from 'react';

import { styled } from 'styled-components';

import { Fieldset } from '@navikt/ds-react';
import { Fieldset, TextField } from '@navikt/ds-react';
import { ASpacing6 } from '@navikt/ds-tokens/dist/tokens';
import { FamilieInput } from '@navikt/familie-form-elements';
import { Valideringsstatus } from '@navikt/familie-skjema';
import CountrySelect from '@navikt/landvelger';

Expand Down Expand Up @@ -44,30 +43,30 @@ const BrevmottakerSkjema: React.FC<IProps> = ({ preutfyltNavn }) => {
return (
<>
<StyledFieldset legend="Skjema for manuell registrering av brevmottaker" hideLegend>
<FamilieInput
<TextField
{...skjema.felter.navn.hentNavBaseSkjemaProps(skjema.visFeilmeldinger)}
label={'Navn'}
erLesevisning={!!preutfyltNavn}
readOnly={!!preutfyltNavn}
onChange={(event): void => {
skjema.felter.navn.validerOgSettFelt(event.target.value);
}}
/>
<FamilieInput
<TextField
{...skjema.felter.adresselinje1.hentNavBaseSkjemaProps(skjema.visFeilmeldinger)}
label={'Adresselinje 1'}
onChange={(event): void => {
skjema.felter.adresselinje1.validerOgSettFelt(event.target.value);
}}
/>
<FamilieInput
<TextField
{...skjema.felter.adresselinje2.hentNavBaseSkjemaProps(skjema.visFeilmeldinger)}
label={'Adresselinje 2 (valgfri)'}
onChange={(event): void => {
skjema.felter.adresselinje2.validerOgSettFelt(event.target.value);
}}
/>
<PostnummerOgStedContainer>
<FamilieInput
<TextField
{...skjema.felter.postnummer.hentNavBaseSkjemaProps(
skjema.visFeilmeldinger
)}
Expand All @@ -76,7 +75,7 @@ const BrevmottakerSkjema: React.FC<IProps> = ({ preutfyltNavn }) => {
skjema.felter.postnummer.validerOgSettFelt(event.target.value);
}}
/>
<FamilieInput
<TextField
{...skjema.felter.poststed.hentNavBaseSkjemaProps(skjema.visFeilmeldinger)}
label={'Poststed'}
onChange={(event): void => {
Expand Down
9 changes: 4 additions & 5 deletions src/frontend/komponenter/Fagsak/Brevmottaker/LeggTilEndreBrevmottakerModal.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,8 @@ import React from 'react';

import { styled } from 'styled-components';

import { Button, Fieldset, Modal, Radio, RadioGroup, Select } from '@navikt/ds-react';
import { Button, Fieldset, Modal, Radio, RadioGroup, Select, TextField } from '@navikt/ds-react';
import { ASpacing6 } from '@navikt/ds-tokens/dist/tokens';
import { FamilieInput } from '@navikt/familie-form-elements';
import { RessursStatus } from '@navikt/familie-typer';

import { useBehandling } from '../../../context/BehandlingContext';
Expand Down Expand Up @@ -178,7 +177,7 @@ export const LeggTilEndreBrevmottakerModal: React.FC = () => {
<BrevmottakerSkjema preutfyltNavn={preutfyltNavn} />
)}
{adresseKilde === AdresseKilde.OPPSLAG_REGISTER && (
<FamilieInput
<TextField
{...skjema.felter.fødselsnummer.hentNavBaseSkjemaProps(
skjema.visFeilmeldinger
)}
Expand All @@ -192,7 +191,7 @@ export const LeggTilEndreBrevmottakerModal: React.FC = () => {
)}
{adresseKilde === AdresseKilde.OPPSLAG_ORGANISASJONSREGISTER && (
<>
<FamilieInput
<TextField
{...skjema.felter.organisasjonsnummer.hentNavBaseSkjemaProps(
skjema.visFeilmeldinger
)}
Expand All @@ -203,7 +202,7 @@ export const LeggTilEndreBrevmottakerModal: React.FC = () => {
);
}}
/>
<FamilieInput
<TextField
{...skjema.felter.navn.hentNavBaseSkjemaProps(
skjema.visFeilmeldinger
)}
Expand Down
126 changes: 70 additions & 56 deletions src/frontend/komponenter/Fagsak/Fakta/FaktaPeriode/FeilutbetalingFaktaPeriodeSkjema.tsx
View file
Open in desktop
Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lesevisning
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Før
image

Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ import * as React from 'react';

import classNames from 'classnames';

import { BodyShort, Table } from '@navikt/ds-react';
import { FamilieSelect } from '@navikt/familie-form-elements';
import { BodyShort, Select, Table, VStack } from '@navikt/ds-react';

import {
hendelsetyper,
Expand All @@ -15,6 +14,12 @@ import {
import { formatterDatostring, formatCurrencyNoKr } from '../../../../utils';
import { useFeilutbetalingFakta } from '../FeilutbetalingFaktaContext';
import { FaktaPeriodeSkjemaData } from '../typer/feilutbetalingFakta';
import { styled } from 'styled-components';
import { ASpacing1 } from '@navikt/ds-tokens/dist/tokens';

const StyledVStack = styled(VStack)`
margin-top: ${ASpacing1};
`;

interface IProps {
periode: FaktaPeriodeSkjemaData;
Expand Down Expand Up @@ -61,61 +66,70 @@ const FeilutbetalingFaktaPeriode: React.FC<IProps> = ({
)} - ${formatterDatostring(periode.periode.tom)}`}</BodyShort>
</Table.DataCell>
<Table.DataCell>
<FamilieSelect
id={`perioder.${index}.årsak`}
data-testid={`perioder.${index}.årsak`}
label={<span className="sr-only">Årsak</span>}
onChange={event => onChangeÅrsak(event)}
value={periode.hendelsestype || '-'}
erLesevisning={erLesevisning}
lesevisningVerdi={
periode.hendelsestype ? hendelsetyper[periode.hendelsestype] : ''
}
error={
visFeilmeldinger &&
feilmeldinger?.find(
meld => meld.periode === periode.index && meld.gjelderHendelsetype
)?.melding
}
size={erLesevisning ? 'medium' : 'small'}
>
<option>-</option>
{hendelseTyper?.map(type => (
<option key={type} value={type}>
{hendelsetyper[type]}
</option>
))}
</FamilieSelect>
{hendelseUnderTyper && hendelseUnderTyper.length > 0 && (
<FamilieSelect
id={`perioder.${index}.underårsak`}
data-testid={`perioder.${index}.underårsak`}
label={<span className="sr-only">Underårsak</span>}
onChange={event => onChangeUnderÅrsak(event)}
value={periode.hendelsesundertype || '-'}
erLesevisning={erLesevisning}
lesevisningVerdi={
periode.hendelsesundertype
? hendelseundertyper[periode.hendelsesundertype]
: ''
}
error={
visFeilmeldinger &&
feilmeldinger?.find(
meld =>
meld.periode === periode.index && meld.gjelderHendelseundertype
)?.melding
}
size={erLesevisning ? 'medium' : 'small'}
>
<option>-</option>
{hendelseUnderTyper.map(type => (
<option key={type} value={type}>
{hendelseundertyper[type]}
</option>
<StyledVStack gap="1">
{erLesevisning ? (
halvorbmundal marked this conversation as resolved.
Show resolved Hide resolved
<BodyShort size="small">
{periode.hendelsestype && hendelsetyper[periode.hendelsestype]}
</BodyShort>
) : (
<Select
id={`perioder.${index}.årsak`}
data-testid={`perioder.${index}.årsak`}
label={'Årsak'}
hideLabel
onChange={event => onChangeÅrsak(event)}
value={periode.hendelsestype || '-'}
error={
visFeilmeldinger &&
feilmeldinger?.find(
meld =>
meld.periode === periode.index && meld.gjelderHendelsetype
)?.melding
}
size={'small'}
>
<option>-</option>
{hendelseTyper?.map(type => (
<option key={type} value={type}>
{hendelsetyper[type]}
</option>
))}
</Select>
)}
{hendelseUnderTyper &&
hendelseUnderTyper.length > 0 &&
(erLesevisning ? (
<BodyShort size="small">
{periode.hendelsesundertype &&
hendelseundertyper[periode.hendelsesundertype]}
</BodyShort>
) : (
<Select
id={`perioder.${index}.underårsak`}
data-testid={`perioder.${index}.underårsak`}
label={'Underårsak'}
hideLabel
onChange={event => onChangeUnderÅrsak(event)}
value={periode.hendelsesundertype || '-'}
error={
visFeilmeldinger &&
feilmeldinger?.find(
meld =>
meld.periode === periode.index &&
meld.gjelderHendelseundertype
)?.melding
}
size={'small'}
>
<option>-</option>
{hendelseUnderTyper.map(type => (
<option key={type} value={type}>
{hendelseundertyper[type]}
</option>
))}
</Select>
))}
</FamilieSelect>
)}
</StyledVStack>
</Table.DataCell>
<Table.DataCell align="right" className={classNames('redText')}>
<BodyShort size="small">{formatCurrencyNoKr(periode.feilutbetaltBeløp)}</BodyShort>
Expand Down
17 changes: 5 additions & 12 deletions ...komponenter/Fagsak/Foreldelse/ForeldelsePeriode/FeilutbetalingForeldelsePeriodeSkjema.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,8 @@ import { styled } from 'styled-components';
import { Column, Row } from 'nav-frontend-grid';

import { ExternalLinkIcon } from '@navikt/aksel-icons';
import { BodyLong, Heading, Link, Radio, ReadMore, Textarea } from '@navikt/ds-react';
import { BodyLong, Heading, Link, Radio, RadioGroup, ReadMore, Textarea } from '@navikt/ds-react';
import { ABorderStrong, ASpacing3 } from '@navikt/ds-tokens/dist/tokens';
import { FamilieRadioGruppe } from '@navikt/familie-form-elements';
import { Valideringsstatus } from '@navikt/familie-skjema';

import {
Expand Down Expand Up @@ -170,17 +169,11 @@ const FeilutbetalingForeldelsePeriodeSkjema: React.FC<IProps> = ({
<Spacer20 />
<Row>
<Column md="5">
<FamilieRadioGruppe
<RadioGroup
id="foreldet"
erLesevisning={erLesevisning}
readOnly={erLesevisning}
legend="Vurder om perioden er foreldet"
value={
!erLesevisning
? skjema.felter.foreldelsesvurderingstype.verdi
: periode.foreldelsesvurderingstype
? foreldelsevurderinger[periode.foreldelsesvurderingstype]
: ''
}
value={skjema.felter.foreldelsesvurderingstype.verdi}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

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

error={
ugyldigVurderingValgt
? skjema.felter.foreldelsesvurderingstype.feilmelding?.toString()
Expand All @@ -195,7 +188,7 @@ const FeilutbetalingForeldelsePeriodeSkjema: React.FC<IProps> = ({
{foreldelsevurderinger[type]}
</Radio>
))}
</FamilieRadioGruppe>
</RadioGroup>
</Column>
<Column md="5">
{erMedTilleggsfrist && (
Expand Down
Loading
Loading