Skip to content

Commit

Permalink
Fjern vis komponent (#2893)
Browse files Browse the repository at this point in the history
  • Loading branch information
@heyvard
heyvard authored Oct 22, 2024
1 parent e688903 commit 3c634b8
Show file tree
Hide file tree
Showing 33 changed files with 347 additions and 507 deletions.
4 changes: 1 addition & 3 deletions src/components/animate-on-mount.tsx
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
import React, { useEffect, useRef, useState } from 'react'

import Vis from './vis'

interface AnimateOnMountProps {
mounted: boolean
enter: string
Expand All @@ -26,7 +24,7 @@ const AnimateOnMount = (props: AnimateOnMountProps) => {

return (
<div ref={animRef} className={`${start} ${styles}`}>
<Vis hvis={mounted} render={() => children} />
{mounted && children}
</div>
)
}
Expand Down
16 changes: 6 additions & 10 deletions src/components/avbrutt/avbrutt-soknad.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ import FristSykepenger from '../frist-sykepenger/frist-sykepenger'
import { GjenstaendeSoknader, hentGjenstaendeSoknader } from '../gjenstaende-soknader/gjenstaende-soknader'
import Opplysninger from '../opplysninger-fra-sykmelding/opplysninger'
import { urlTilSoknad } from '../soknad/soknad-link'
import Vis from '../vis'
import useSoknader from '../../hooks/useSoknader'
import QueryStatusPanel from '../queryStatusPanel/QueryStatusPanel'
import { soknadBreadcrumb, useUpdateBreadcrumbs } from '../../hooks/useBreadcrumbs'
Expand Down Expand Up @@ -84,15 +83,12 @@ const AvbruttSoknad = () => {
</BodyLong>
</>
)}
<Vis
hvis={dayjs(valgtSoknad.avbruttDato).isAfter(dayjs().subtract(2, 'seconds'))}
render={() => (
<GjenstaendeSoknader
style={{ marginTop: '1rem', marginBottom: '1rem' }}
soknader={gjenstaendeSoknader}
/>
)}
/>
{dayjs(valgtSoknad.avbruttDato).isAfter(dayjs().subtract(2, 'seconds')) && (
<GjenstaendeSoknader
style={{ marginTop: '1rem', marginBottom: '1rem' }}
soknader={gjenstaendeSoknader}
/>
)}
<GjenapneSoknad />
</>
)
Expand Down
7 changes: 2 additions & 5 deletions src/components/avbrutt/gjenapneknapp.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import React from 'react'

import { logEvent } from '../amplitude/amplitude'
import { useGjenapne } from '../../hooks/useGjenapne'
import Vis from '../vis'
import { useSoknadMedDetaljer } from '../../hooks/useSoknadMedDetaljer'

const GjenapneSoknad = () => {
Expand All @@ -15,10 +14,8 @@ const GjenapneSoknad = () => {

return (
<>
<Vis
hvis={gjenapneError}
render={() => <Alert variant="error">Beklager, klarte ikke gjenåpne søknaden din</Alert>}
/>
{gjenapneError && <Alert variant="error">Beklager, klarte ikke gjenåpne søknaden din</Alert>}

<Button
data-cy="bruk-soknad-likevel"
variant="tertiary"
Expand Down
8 changes: 3 additions & 5 deletions src/components/ettersending/ettersending.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import React, { useState } from 'react'

import { tekst } from '../../utils/tekster'
import { tekstMedHtml } from '../../utils/html-react-parser-utils'
import Vis from '../vis'
import { useEttersendArbeidsgiver } from '../../hooks/useEttersendArbeidsgiver'
import { useSoknadMedDetaljer } from '../../hooks/useSoknadMedDetaljer'

Expand Down Expand Up @@ -61,10 +60,9 @@ const Ettersending = ({ gjelder }: EttersendingProps) => {
>
<Modal.Body>
<BodyShort spacing>{tekstMedHtml(tekst('kvittering.info.send-til-arbeidsgiver'))}</BodyShort>
<Vis
hvis={ettersendArbeidsgiverError}
render={() => <Alert variant="error">Beklager, klarte ikke ettersende søknaden din</Alert>}
/>
{ettersendArbeidsgiverError && (
<Alert variant="error">Beklager, klarte ikke ettersende søknaden din</Alert>
)}
<Button
type="button"
size="small"
Expand Down
8 changes: 3 additions & 5 deletions src/components/feil/feil-lokal.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import { useFormContext } from 'react-hook-form'

import { Sporsmal } from '../../types/types'
import { hentFeilmelding } from '../sporsmal/sporsmal-utils'
import Vis from '../vis'

interface FeilProps {
sporsmal: Sporsmal
Expand All @@ -17,16 +16,15 @@ const FeilLokal = ({ sporsmal }: FeilProps) => {
const feilmelding = hentFeilmelding(sporsmal, errors[sporsmal.id])

return (
<Vis
hvis={errors[sporsmal.id]}
render={() => (
<>
{errors[sporsmal.id] && (
<div role="alert" aria-live="assertive">
<BodyShort as="span" className="mt-2 block font-bold text-surface-danger" data-cy="feil-lokal">
{feilmelding.lokal}
</BodyShort>
</div>
)}
/>
</>
)
}

Expand Down
57 changes: 23 additions & 34 deletions src/components/feil/feil-oppsummering.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import { RSSvartype } from '../../types/rs-types/rs-svartype'
import { Soknad, Sporsmal } from '../../types/types'
import { flattenSporsmal } from '../../utils/soknad-utils'
import { logEvent } from '../amplitude/amplitude'
import Vis from '../vis'
import { FetchError } from '../../utils/fetch'

const FeilOppsummering = ({
Expand Down Expand Up @@ -95,12 +94,18 @@ const FeilOppsummering = ({
klikk()
}
}

return (
<div aria-live="polite" role="alert">
<Vis
hvis={antall > 0}
render={() => {
const elements = entries.map((list) => (
{antall > 0 && (
<ErrorSummary
ref={oppsummering}
size="medium"
heading={sendError ? 'Beklager, det oppstod en feil' : 'Det er ' + antall + ' feil i skjemaet'}
className="mt-8"
data-cy="feil-oppsumering"
>
{entries.map((list) => (
<ErrorSummary.Item
href="#"
key={list[1].message + list[0]}
Expand All @@ -110,36 +115,20 @@ const FeilOppsummering = ({
>
{list[1].message}
</ErrorSummary.Item>
))
if (sendError) {
elements.push(
<ErrorSummary.Item
onKeyDown={(e) => handleKeyDownSendError(e)}
onClick={() => klikk()}
key="send-error"
>
{sendError?.status == 400
? 'Vi har lagret dine svar, men du må laste inn siden på nytt før du kan sende søknaden. Klikk her for å laste inn siden på nytt.'
: 'Beklager, det oppstod en teknisk feil.'}
</ErrorSummary.Item>,
)
}
const heading = sendError
? 'Beklager, det oppstod en feil'
: 'Det er ' + antall + ' feil i skjemaet'
return (
<ErrorSummary
ref={oppsummering}
size="medium"
heading={heading}
className="mt-8"
data-cy="feil-oppsumering"
))}
{sendError && (
<ErrorSummary.Item
onKeyDown={(e) => handleKeyDownSendError(e)}
onClick={() => klikk()}
key="send-error"
>
{elements}
</ErrorSummary>
)
}}
/>
{sendError?.status == 400
? 'Vi har lagret dine svar, men du må laste inn siden på nytt før du kan sende søknaden. Klikk her for å laste inn siden på nytt.'
: 'Beklager, det oppstod en teknisk feil.'}
</ErrorSummary.Item>
)}
</ErrorSummary>
)}
</div>
)
}
Expand Down
78 changes: 37 additions & 41 deletions src/components/filopplaster/fil-liste/fil-liste.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import { Kvittering } from '../../../types/types'
import { getLedetekst, tekst } from '../../../utils/tekster'
import { formatterTall } from '../../../utils/utils'
import { hentSvar } from '../../sporsmal/hent-svar'
import Vis from '../../vis'
import { useSoknadMedDetaljer } from '../../../hooks/useSoknadMedDetaljer'

import KvitteringListeVisning from './kvittering-listevisning'
Expand All @@ -30,47 +29,44 @@ const FilListe = () => {
: 0.0) / 100

return (
<Vis
hvis={kvitteringer.length > 0}
render={() => (
<>
<Table zebraStripes={true}>
<Table.Header>
<Table.Row>
<Table.HeaderCell scope="col">
<span className="sr-only">Ekspander for å se kvitteringen</span>
</Table.HeaderCell>
<Table.HeaderCell scope="col">Utgift</Table.HeaderCell>
<Table.HeaderCell scope="col">Beløp</Table.HeaderCell>
<Table.HeaderCell scope="col">
<span className="sr-only">Slett kvitteringen</span>
</Table.HeaderCell>
</Table.Row>
</Table.Header>
<Table.Body>
{kvitteringer.reverse().map((kvittering: Kvittering) => (
<KvitteringListeVisning
key={kvittering.blobId}
kvittering={kvittering}
sporsmal={sporsmal}
/>
))}
<Table.Row>
<Table.DataCell colSpan={2} className="border-b-0 font-bold">
{getLedetekst(tekst('fil_liste.utlegg.sum'), {
'%ANTALL_BILAG%': kvitteringer.length,
'%FLERTALL%': kvitteringer.length > 1 ? 'er' : '',
})}
</Table.DataCell>
<Table.DataCell colSpan={2} className="border-b-0 font-bold">
{formatterTall(totaltBeløp())} kr
</Table.DataCell>
</Table.Row>
</Table.Body>
</Table>
</>
<>
{kvitteringer.length > 0 && (
<Table zebraStripes={true}>
<Table.Header>
<Table.Row>
<Table.HeaderCell scope="col">
<span className="sr-only">Ekspander for å se kvitteringen</span>
</Table.HeaderCell>
<Table.HeaderCell scope="col">Utgift</Table.HeaderCell>
<Table.HeaderCell scope="col">Beløp</Table.HeaderCell>
<Table.HeaderCell scope="col">
<span className="sr-only">Slett kvitteringen</span>
</Table.HeaderCell>
</Table.Row>
</Table.Header>
<Table.Body>
{kvitteringer.reverse().map((kvittering: Kvittering) => (
<KvitteringListeVisning
key={kvittering.blobId}
kvittering={kvittering}
sporsmal={sporsmal}
/>
))}
<Table.Row>
<Table.DataCell colSpan={2} className="border-b-0 font-bold">
{getLedetekst(tekst('fil_liste.utlegg.sum'), {
'%ANTALL_BILAG%': kvitteringer.length,
'%FLERTALL%': kvitteringer.length > 1 ? 'er' : '',
})}
</Table.DataCell>
<Table.DataCell colSpan={2} className="border-b-0 font-bold">
{formatterTall(totaltBeløp())} kr
</Table.DataCell>
</Table.Row>
</Table.Body>
</Table>
)}
/>
</>
)
}

Expand Down
40 changes: 15 additions & 25 deletions src/components/frist-sykepenger/hvorfor-soknad-sykepenger.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import React from 'react'

import { RSSoknadstype } from '../../types/rs-types/rs-soknadstype'
import { tekst } from '../../utils/tekster'
import Vis from '../vis'

interface HvorforSoknadSykepengerProps {
soknadstype: RSSoknadstype
Expand All @@ -16,32 +15,23 @@ const HvorforSoknadSykepenger = ({ soknadstype }: HvorforSoknadSykepengerProps)
{tekst('hvorfor-soknad-sykepenger.overskrift')}
</Heading>

<Vis
hvis={soknadstype === RSSoknadstype.REISETILSKUDD}
render={() => <BodyShort>{tekst('hvorfor-soknad-reisetilskudd')}</BodyShort>}
/>
{soknadstype === RSSoknadstype.REISETILSKUDD && (
<BodyShort>{tekst('hvorfor-soknad-reisetilskudd')}</BodyShort>
)}

<Vis
hvis={soknadstype === RSSoknadstype.GRADERT_REISETILSKUDD}
render={() => (
<>
<BodyShort spacing>{tekst('hvorfor-soknad-gradertreisetilskudd')}</BodyShort>
<BodyShort>{tekst('hvorfor-soknad-sykepenger.jobba')}</BodyShort>
</>
)}
/>
{soknadstype === RSSoknadstype.GRADERT_REISETILSKUDD && (
<>
<BodyShort spacing>{tekst('hvorfor-soknad-gradertreisetilskudd')}</BodyShort>
<BodyShort>{tekst('hvorfor-soknad-sykepenger.jobba')}</BodyShort>
</>
)}

<Vis
hvis={
soknadstype !== RSSoknadstype.REISETILSKUDD && soknadstype !== RSSoknadstype.GRADERT_REISETILSKUDD
}
render={() => (
<>
<BodyShort spacing>{tekst('hvorfor-soknad-sykepenger')}</BodyShort>
<BodyShort>{tekst('hvorfor-soknad-sykepenger.jobba')}</BodyShort>
</>
)}
/>
{soknadstype !== RSSoknadstype.REISETILSKUDD && soknadstype !== RSSoknadstype.GRADERT_REISETILSKUDD && (
<>
<BodyShort spacing>{tekst('hvorfor-soknad-sykepenger')}</BodyShort>
<BodyShort>{tekst('hvorfor-soknad-sykepenger.jobba')}</BodyShort>
</>
)}
</>
)
}
Expand Down
8 changes: 2 additions & 6 deletions src/components/kvittering/alle-andre.tsx
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import React from 'react'

import { RSSoknadstype } from '../../types/rs-types/rs-soknadstype'
import Vis from '../vis'
import { useSoknadMedDetaljer } from '../../hooks/useSoknadMedDetaljer'

import KvitteringVidere from './innhold/kvittering-videre'
Expand All @@ -16,11 +15,8 @@ const AlleAndre = () => {
return (
<>
<KvitteringStatus />
<Vis hvis={valgtSoknad.soknadstype !== RSSoknadstype.OPPHOLD_UTLAND} render={() => <KvitteringVidere />} />
<Vis
hvis={valgtSoknad.soknadstype === RSSoknadstype.OPPHOLD_UTLAND}
render={() => <KvitteringUtenlands />}
/>
{valgtSoknad.soknadstype !== RSSoknadstype.OPPHOLD_UTLAND && <KvitteringVidere />}
{valgtSoknad.soknadstype === RSSoknadstype.OPPHOLD_UTLAND && <KvitteringUtenlands />}
</>
)
}
Expand Down
Loading

0 comments on commit 3c634b8

Please sign in to comment.