From 42b9243c87622ecb2fc3553ec3f4214b43b43f74 Mon Sep 17 00:00:00 2001
From: Matthew Stankiewicz <stankm@pointclickcare.com>
Date: Wed, 28 Feb 2024 17:31:21 -0500
Subject: [PATCH] ui: reformat participants edit/add styling

---
 .../sed/edit/sed-participants-edit.jsx        | 65 +++++++++++--------
 1 file changed, 37 insertions(+), 28 deletions(-)

diff --git a/src/app/components/listing/details/sed/edit/sed-participants-edit.jsx b/src/app/components/listing/details/sed/edit/sed-participants-edit.jsx
index 8926b9434d..003cf19271 100755
--- a/src/app/components/listing/details/sed/edit/sed-participants-edit.jsx
+++ b/src/app/components/listing/details/sed/edit/sed-participants-edit.jsx
@@ -5,6 +5,8 @@ import {
   AccordionSummary,
   Box,
   Button,
+  CardContent,
+  IconButton,
   Table,
   TableBody,
   TableCell,
@@ -15,6 +17,7 @@ import {
 } from '@material-ui/core';
 import AddIcon from '@material-ui/icons/Add';
 import ExpandMoreIcon from '@material-ui/icons/ExpandMore';
+import RemoveCircleOutlineIcon from '@material-ui/icons/RemoveCircleOutline';
 import { object } from 'prop-types';
 
 import ChplSedParticipantAdd from './sed-participant-add';
@@ -154,7 +157,7 @@ function ChplSedParticipantsEdit({ task: initialTask }) {
             <TableHead>
               <TableRow>
                 <TableCell size="medium">Occupation</TableCell>
-                <TableCell size="medium">Education Type</TableCell>
+                <TableCell size="medium" style={{ minWidth: 150 }}>Education Type</TableCell>
                 <TableCell size="medium">Product Experience (Months)</TableCell>
                 <TableCell size="medium">Professional Experience (Months)</TableCell>
                 <TableCell size="medium">Computer Experience (Months)</TableCell>
@@ -169,7 +172,7 @@ function ChplSedParticipantsEdit({ task: initialTask }) {
                 .map((participant) => (
                   <TableRow key={participant.id ?? participant.uniqueId}>
                     <TableCell>{ participant.occupation }</TableCell>
-                    <TableCell>{ participant.educationType.name }</TableCell>
+                    <TableCell style={{ minWidth: 150 }}>{ participant.educationType.name }</TableCell>
                     <TableCell>{ participant.productExperienceMonths }</TableCell>
                     <TableCell>{ participant.professionalExperienceMonths }</TableCell>
                     <TableCell>{ participant.computerExperienceMonths }</TableCell>
@@ -179,45 +182,51 @@ function ChplSedParticipantsEdit({ task: initialTask }) {
                     <TableCell>
                       { task.testParticipants.some((p) => ((p.id ?? p.uniqueId) === (participant.id ?? participant.uniqueId)))
                         ? (
-                          <Button
+                          <IconButton
                             onClick={() => remove(participant)}
                           >
-                            Remove
-                          </Button>
+                            <RemoveCircleOutlineIcon color="error" />
+                          </IconButton>
                         ) : (
-                          <Button
+                          <IconButton
                             onClick={() => add(participant)}
                           >
-                            Add
-                          </Button>
+                            <AddIcon color="primary" />
+                          </IconButton>
                         )}
                     </TableCell>
                   </TableRow>
                 ))}
             </TableBody>
           </Table>
-          { !addingParticipant
-            && (
-              <Box paddingX={2} paddingY={2}>
-                <Button
-                  size="medium"
-                  color="primary"
-                  variant="outlined"
-                  onClick={() => setAddingParticipant(true)}
-                  endIcon={<AddIcon fontSize="medium" />}
-                >
-                  Add Test Participant
-                </Button>
-              </Box>
-            )}
-          { addingParticipant
-            && (
-              <ChplSedParticipantAdd
-                dispatch={handleDispatch}
-              />
-            )}
         </>
       </AccordionDetails>
+      <>
+        { !addingParticipant
+          && (
+            <Box paddingX={2} paddingY={2}>
+              <Button
+                size="medium"
+                color="primary"
+                variant="outlined"
+                onClick={() => setAddingParticipant(true)}
+                endIcon={<AddIcon fontSize="medium" />}
+              >
+                Add Test Participant
+              </Button>
+            </Box>
+          )}
+        { addingParticipant
+          && (
+            <>
+              <CardContent>
+                <ChplSedParticipantAdd
+                  dispatch={handleDispatch}
+                />
+              </CardContent>
+            </>
+          )}
+      </>
     </Accordion>
   );
 }