Remove redundant fragments

navikt · Aug 29, 2023 · 9efb571 · 9efb571
1 parent 55bc297
commit 9efb571
Show file tree

Hide file tree

Showing 4 changed files with 64 additions and 71 deletions.
diff --git a/apps/frontend/src/components/team/ModalContactAllTeams.tsx b/apps/frontend/src/components/team/ModalContactAllTeams.tsx
@@ -104,43 +104,41 @@ const getContactAddress = async (productTeam: ProductTeamResponse) => {
 export const ModalContactAllTeams = (properties: ModalTeamProperties) => {
   const { onClose, title, isOpen, teams } = properties;
   return (
-    <>
-      <Modal className={styles.modalStyles} header={{ heading: title }} onClose={onClose} open={isOpen}>
-        <Modal.Body>
-          <Detail
-            className={css`
-              font-size: 16px;
-            `}
+    <Modal className={styles.modalStyles} header={{ heading: title }} onClose={onClose} open={isOpen}>
+      <Modal.Body>
+        <Detail
+          className={css`
+            font-size: 16px;
+          `}
+        >
+          Hvis "Åpne e-postklient" knappen ikke fungerer bruk "Kopier e-poster" knappen og lim disse inn i din
+          e-postklient
+        </Detail>
+        <div
+          className={css`
+            display: flex;
+            flex-direction: column;
+            align-items: center;
+          `}
+        >
+          <Button
+            className={styles.buttonStyle}
+            onClick={async () => {
+              await contactTeamsOutlook(teams);
+            }}
           >
-            Hvis "Åpne e-postklient" knappen ikke fungerer bruk "Kopier e-poster" knappen og lim disse inn i din
-            e-postklient
-          </Detail>
-          <div
-            className={css`
-              display: flex;
-              flex-direction: column;
-              align-items: center;
-            `}
+            Åpne e-postklient
+          </Button>
+          <Button
+            className={styles.buttonStyle}
+            onClick={async () => {
+              await contactTeamsCopy(teams);
+            }}
           >
-            <Button
-              className={styles.buttonStyle}
-              onClick={async () => {
-                await contactTeamsOutlook(teams);
-              }}
-            >
-              Åpne e-postklient
-            </Button>
-            <Button
-              className={styles.buttonStyle}
-              onClick={async () => {
-                await contactTeamsCopy(teams);
-              }}
-            >
-              Kopier e-poster
-            </Button>
-          </div>
-        </Modal.Body>
-      </Modal>
-    </>
+            Kopier e-poster
+          </Button>
+        </div>
+      </Modal.Body>
+    </Modal>
   );
 };
diff --git a/apps/frontend/src/components/team/ShortSummarySection.tsx b/apps/frontend/src/components/team/ShortSummarySection.tsx
@@ -90,9 +90,7 @@ function TeamOwner(properties: { teamOwner?: Resource; area?: ProductArea }) {
   const teamOwner = properties.teamOwner;
 
   return (
-    <>
-      <TextWithLabel label="Teameier" text={teamOwner ? <TeamOwnerResource resource={teamOwner} /> : "Ingen eier"} />
-    </>
+    <TextWithLabel label="Teameier" text={teamOwner ? <TeamOwnerResource resource={teamOwner} /> : "Ingen eier"} />
   );
 }
 

diff --git a/apps/frontend/src/pages/area/ModalArea.tsx b/apps/frontend/src/pages/area/ModalArea.tsx
@@ -379,30 +379,29 @@ export const ModalArea = (properties: ModalAreaProperties) => {
                 `}
               >
                 {resourceList.map((rl) => (
-                  <>
-                    <div
-                      className={css`
-                        display: flex;
-                        justify-content: space-between;
-                        align-items: center;
-                        margin-left: 50%;
-                        margin-bottom: 1rem;
-                      `}
-                    >
-                      <BodyShort size="medium">
-                        <b>{rl.fullName}</b> - ({rl.navIdent})
-                      </BodyShort>
-                      <Button
-                        icon={<TrashIcon aria-hidden />}
-                        onClick={() => {
-                          const newArray = resourceList.filter((r) => r.navIdent !== rl.navIdent);
-                          setResourceList([...newArray]);
-                        }}
-                        size="small"
-                        variant="tertiary"
-                      ></Button>
-                    </div>
-                  </>
+                  <div
+                    className={css`
+                      display: flex;
+                      justify-content: space-between;
+                      align-items: center;
+                      margin-left: 50%;
+                      margin-bottom: 1rem;
+                    `}
+                    key={rl.navIdent}
+                  >
+                    <BodyShort size="medium">
+                      <b>{rl.fullName}</b> - ({rl.navIdent})
+                    </BodyShort>
+                    <Button
+                      icon={<TrashIcon aria-hidden />}
+                      onClick={() => {
+                        const newArray = resourceList.filter((r) => r.navIdent !== rl.navIdent);
+                        setResourceList([...newArray]);
+                      }}
+                      size="small"
+                      variant="tertiary"
+                    ></Button>
+                  </div>
                 ))}
               </div>
             </div>

diff --git a/apps/frontend/src/pages/location/LocationPage.tsx b/apps/frontend/src/pages/location/LocationPage.tsx
@@ -86,14 +86,12 @@ export const LocationPage = () => {
   return (
     <Fragment>
       {!loading && locationBuilding && locationStats && (
-        <>
-          <BuildingInfo
-            locationBuilding={locationBuilding}
-            locationCode={parameters.locationCode || ""}
-            locationStats={locationStats.locationSummaryMap}
-            sectionList={locationBuilding.subLocations || []}
-          />
-        </>
+        <BuildingInfo
+          locationBuilding={locationBuilding}
+          locationCode={parameters.locationCode || ""}
+          locationStats={locationStats.locationSummaryMap}
+          sectionList={locationBuilding.subLocations || []}
+        />
       )}
       {parameters.locationCode && locationSection && locationStats && !loading && (
         <BuildingFloors locationStats={locationStats.locationSummaryMap} section={locationSection} />