fix: paysubsidy attachment (HL-1009)

City-of-Helsinki · Oct 10, 2023 · de60caa · de60caa
1 parent 42cbeba
commit de60caa
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 23 deletions.
diff --git a/frontend/benefit/handler/src/components/attachmentsListView/AttachmentsListView.tsx b/frontend/benefit/handler/src/components/attachmentsListView/AttachmentsListView.tsx
@@ -1,5 +1,5 @@
 import { ATTACHMENT_TYPES } from 'benefit-shared/constants';
-import { IconLinkExternal, IconPaperclip } from 'hds-react';
+import { IconPaperclip } from 'hds-react';
 import * as React from 'react';
 import {
   $ViewField,
@@ -37,33 +37,25 @@ const AttachmentsListView: React.FC<AttachmentsListViewProps> = ({
     <div>
       {attachmentItems.length > 0 && (
         <$GridCell $colStart={1} $colSpan={6}>
-          {title && (
-            <$ViewFieldBold css={{ fontSize: theme.fontSize.body.m }}>
-              {title}
-            </$ViewFieldBold>
-          )}
+          {title && <$ViewFieldBold>{title}</$ViewFieldBold>}
           {attachmentItems.map((attachment) => (
             <$ViewField
+              onClick={() => handleOpenFile(attachment)}
+              aria-label={`${attachment.attachmentFileName}_open`}
               css={{
                 display: 'flex',
                 alignItems: 'center',
                 marginTop: `${theme.spacing.xs}`,
                 fontSize: theme.fontSize.body.m,
                 color: theme.colors.coatOfArms,
+                cursor: 'pointer',
               }}
               key={attachment.attachmentFileName}
             >
-              <IconPaperclip
-                css={{ color: theme.colors.black }}
-                aria-label={attachment.attachmentFileName}
-              />
-              {attachment.attachmentFileName}
-              <IconLinkExternal
-                onClick={() => handleOpenFile(attachment)}
-                aria-label={`${attachment.attachmentFileName}_open`}
-                css={{ marginLeft: theme.spacing.xs, cursor: 'pointer' }}
-                size="s"
-              />
+              <IconPaperclip aria-label={attachment.attachmentFileName} />
+              <span css={{ textDecoration: 'underline' }}>
+                {attachment.attachmentFileName}
+              </span>
             </$ViewField>
           ))}
         </$GridCell>

diff --git a/frontend/benefit/handler/src/components/newApplication/formContent/FormContent.tsx b/frontend/benefit/handler/src/components/newApplication/formContent/FormContent.tsx
@@ -91,6 +91,11 @@ const FormContent: React.FC<Props> = ({
     getErrorMessage,
   } = useFormContent(formik, fields);
 
+  const truthySubsidies = new Set([
+    PAY_SUBSIDY_GRANTED.GRANTED,
+    PAY_SUBSIDY_GRANTED.GRANTED_AGED,
+  ]);
+
   const theme = useTheme();
   useAlertBeforeLeaving(formik.dirty);
 
@@ -513,10 +518,7 @@ const FormContent: React.FC<Props> = ({
             />
           </SelectionGroup>
         </$GridCell>
-        {[
-          PAY_SUBSIDY_GRANTED.GRANTED,
-          PAY_SUBSIDY_GRANTED.GRANTED_AGED,
-        ].includes(formik.values.paySubsidyGranted) && (
+        {truthySubsidies.has(formik.values.paySubsidyGranted) && (
           <$GridCell
             as={$Grid}
             $colSpan={12}
@@ -658,7 +660,7 @@ const FormContent: React.FC<Props> = ({
             />
           </$GridCell>
         )}
-        {formik.values.paySubsidyGranted && (
+        {truthySubsidies.has(formik.values.paySubsidyGranted) && (
           <$GridCell $colSpan={12}>
             <AttachmentsList
               attachments={attachments}

diff --git a/frontend/benefit/handler/src/components/newApplication/useApplicationForm.ts b/frontend/benefit/handler/src/components/newApplication/useApplicationForm.ts
@@ -89,7 +89,7 @@ export const useApplicationForm = (): ExtendedComponentProps => {
 
   React.useEffect(() => {
     if (id) {
-      dispatchStep({ type: 'setActive', payload: 2 });
+      dispatchStep({ type: 'setActive', payload: 1 });
     }
   }, [id, dispatchStep]);