Forhindrer at det blir dobbelt opp i prosess-soknadsfrist

navikt · Sep 19, 2024 · 21554ae · 21554ae
1 parent e74b2a8
commit 21554ae
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 36 deletions.
diff --git a/packages/prosess-vilkar-soknadsfrist-v2/src/components/SoknadsfristVilkarForm.tsx b/packages/prosess-vilkar-soknadsfrist-v2/src/components/SoknadsfristVilkarForm.tsx
@@ -191,60 +191,38 @@ export const SoknadsfristVilkarForm = ({
   const AksjonspunktText = () => {
     if (harLøstManueltAksjonspunkt && !editForm) {
       return (
-        <Label size="small" as="p">
+        <Label className="mt-2" size="small" as="p">
           Vurder om søknadsfristvilkåret er oppfylt
         </Label>
       );
     }
     if (!isReadOnly) {
       if (harÅpentAksjonspunkt || editForm) {
         return (
-          <AksjonspunktHelpText isAksjonspunktOpen>Vurder om søknadsfristvilkåret er oppfylt</AksjonspunktHelpText>
+          <div className="mt-2">
+            <AksjonspunktHelpText isAksjonspunktOpen>Vurder om søknadsfristvilkåret er oppfylt</AksjonspunktHelpText>
+          </div>
+        );
+      }
+      if (erOverstyrt) {
+        return (
+          <Label className="mt-2" size="small" as="p">
+            Manuell overstyring av automatisk vurdering
+          </Label>
         );
       }
-      return (
-        <Label size="small" as="p">
-          Manuell overstyring av automatisk vurdering
-        </Label>
-      );
     }
     return undefined;
   };
 
   return (
     <Form formMethods={formMethods} onSubmit={handleSubmit}>
-      {!erOverstyrt && dokumenterIAktivPeriode.length > 0 && !editForm && (
-        <div>
-          {Array.isArray(alleDokumenter) &&
-            alleDokumenter.length > 0 &&
-            alleDokumenter.map((field, index) => {
-              const dokument = alleDokumenter.find(dok => dok.journalpostId === field.journalpostId);
-              const documentHash = hash(dokument);
-              return (
-                <SoknadsfristVilkarDokument
-                  key={documentHash}
-                  erAktivtDokument={dokumenterIAktivPeriode.findIndex(d => hash(d) === documentHash) > -1}
-                  skalViseBegrunnelse
-                  readOnly
-                  dokumentIndex={index}
-                  dokument={dokument}
-                  toggleEditForm={toggleEditForm}
-                  dokumentErVurdert={status !== vilkårStatus.IKKE_VURDERT}
-                  periode={periode}
-                  kanEndrePåSøknadsopplysninger={kanEndrePåSøknadsopplysninger}
-                />
-              );
-            })}
-        </div>
-      )}
-
-      {(erOverstyrt || harAksjonspunkt || editForm) && dokumenterIAktivPeriode.length > 0 && (
+      {dokumenterIAktivPeriode.length > 0 && (
         <AksjonspunktBox
           className={styles.aksjonspunktMargin}
           erAksjonspunktApent={erOverstyrt || harÅpentAksjonspunkt || editForm}
         >
-          <AksjonspunktText />
-          <div className="mt-2" />
+          {(erOverstyrt || harÅpentAksjonspunkt || editForm) && <AksjonspunktText />}
           {Array.isArray(alleDokumenter) && alleDokumenter.length > 0
             ? alleDokumenter.map((field, index) => {
                 const dokument = alleDokumenter.find(dok => dok.journalpostId === field.journalpostId);

diff --git a/packages/shared-components/src/aksjonspunktBox.module.css b/packages/shared-components/src/aksjonspunktBox.module.css
@@ -1,8 +1,9 @@
 .aksjonspunkt {
-  padding: 20px;
+  padding: 0 20px 20px;
 }
 
 .aksjonspunkt.erAksjonspunktApent {
+  padding-top: 20px;
   border: 3px solid #fa3;
   border-radius: 4px;
 }