From 98b9a2354b5e718c815c9cc190d73dd9409990aa Mon Sep 17 00:00:00 2001
From: Jen Breese-Kauth <jbreese@stanford.edu>
Date: Fri, 23 Aug 2024 23:23:44 +0000
Subject: [PATCH] fixup for the extra space

---
 .../branch-locations-filtering-table.tsx       | 12 ++++++------
 .../views/sul-people/sul-people-table-view.tsx |  6 +++---
 .../study-place-filtering-table.tsx            | 18 +++++++++---------
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/src/components/views/sul-branch-locations/filtering-table/branch-locations-filtering-table.tsx b/src/components/views/sul-branch-locations/filtering-table/branch-locations-filtering-table.tsx
index cf7deb62..a13aeb15 100644
--- a/src/components/views/sul-branch-locations/filtering-table/branch-locations-filtering-table.tsx
+++ b/src/components/views/sul-branch-locations/filtering-table/branch-locations-filtering-table.tsx
@@ -153,13 +153,13 @@ const TableRow = ({id, imageUrl, path, title, phone, email, mapUrl, address, hou
         </Link>
       </Th>
       <Td className="branch-hours flex w-auto justify-center sm:border-b sm:border-black-40 md:items-center md:justify-start lg:table-cell lg:w-1/4 lg:pr-20 lg:align-middle xl:pr-80">
-        {hoursId ? (
+        {hoursId && (
           <div className="pb-16 lg:pb-0">
             <BranchHours hoursId={hoursId} />
           </div>
-        ) : (
-          ""
         )}
+        {/* Without this, the responsive table library injects a "&nbsp;". */}
+        {""}
       </Td>
       <Td className="block w-auto sm:border-b sm:border-black-40 lg:table-cell lg:w-1/4 lg:pr-20 lg:align-middle xl:pr-80">
         {phone && (
@@ -171,7 +171,7 @@ const TableRow = ({id, imageUrl, path, title, phone, email, mapUrl, address, hou
             {phone}
           </a>
         )}
-        {email ? (
+        {email && (
           <a
             href={`mailto:${email}`}
             className="m-auto mb-16 flex w-fit items-center justify-center gap-4 text-16 font-normal leading-[23px] no-underline hover:bg-black-10 hover:text-brick-dark hover:underline focus:bg-none md:mx-[unset] md:justify-start lg:mb-0"
@@ -179,9 +179,9 @@ const TableRow = ({id, imageUrl, path, title, phone, email, mapUrl, address, hou
             <EnvelopeIcon title="Email" width={20} />
             {email}
           </a>
-        ) : (
-          ""
         )}
+        {/* Without this, the responsive table library injects a "&nbsp;". */}
+        {""}
       </Td>
       <Td className="block w-auto text-16 leading-[23px] sm:border-b sm:border-black-40 lg:table-cell lg:w-1/4 lg:align-middle">
         {address && mapUrl && (
diff --git a/src/components/views/sul-people/sul-people-table-view.tsx b/src/components/views/sul-people/sul-people-table-view.tsx
index a421afc7..421b0bec 100644
--- a/src/components/views/sul-people/sul-people-table-view.tsx
+++ b/src/components/views/sul-people/sul-people-table-view.tsx
@@ -244,13 +244,13 @@ const SulPeopleTableView = ({items, hasHeading}: Props) => {
                   )}
                 </Td>
                 <Td className="block w-auto px-0 py-16 text-center md:border-b md:border-black-40 md:px-9 md:py-16 md:text-left lg:table-cell lg:w-1/5">
-                  {item.libCalId ? (
+                  {item.libCalId && (
                     <div className="pb-16 lg:pb-0">
                       <LibCal libcalId={item.libCalId} srText={item.title} />
                     </div>
-                  ) : (
-                    ""
                   )}
+                  {/* Without this, the responsive table library injects a "&nbsp;". */}
+                  {""}
                 </Td>
               </Tr>
             ))}
diff --git a/src/components/views/sul-study-place/filtering-table/study-place-filtering-table.tsx b/src/components/views/sul-study-place/filtering-table/study-place-filtering-table.tsx
index d6d882e0..293522f1 100644
--- a/src/components/views/sul-study-place/filtering-table/study-place-filtering-table.tsx
+++ b/src/components/views/sul-study-place/filtering-table/study-place-filtering-table.tsx
@@ -210,7 +210,7 @@ const StudyPlaceFilteringTable = ({items}: Props) => {
                 className="block sm:flex-col sm:flex-wrap md:grid md:grid-cols-2 md:grid-rows-[repeat(5,minmax(0,auto))] md:justify-items-start md:gap-x-20 md:text-left lg:!table-row lg:max-h-none"
               >
                 <Td className="table-image m-auto block min-h-fit w-auto place-content-center justify-center sm:border-b sm:border-black-40 md:row-span-5 lg:table-cell lg:min-h-fit lg:w-[125px] lg:pr-32">
-                  {item.branchImageUrl ? (
+                  {item.branchImageUrl && (
                     <div className="relative my-16 block aspect-[3/2] w-auto max-w-[338px] overflow-hidden md:w-[360px] lg:max-w-[125px]">
                       <Image
                         className="object-contain"
@@ -220,9 +220,9 @@ const StudyPlaceFilteringTable = ({items}: Props) => {
                         sizes="(max-width: 992px) 300px, 150px"
                       />
                     </div>
-                  ) : (
-                    ""
                   )}
+                  {/* Without this, the responsive table library injects a "&nbsp;". */}
+                  {""}
                 </Td>
                 <Th
                   scope="row"
@@ -245,13 +245,13 @@ const StudyPlaceFilteringTable = ({items}: Props) => {
                   </Link>
                 </Td>
                 <Td className="justify-left flex w-auto sm:border-b sm:border-black-40 md:text-left lg:table-cell lg:w-1/5 lg:pr-32">
-                  {item.libHours ? (
+                  {item.libHours && (
                     <div className="pb-16 text-16 leading-[23px] lg:pb-0">
                       {item.libHours && <BranchHours hoursId={item.libHours} />}
                     </div>
-                  ) : (
-                    ""
                   )}
+                  {/* Without this, the responsive table library injects a "&nbsp;". */}
+                  {""}
                 </Td>
                 <Td className="block w-auto sm:border-b sm:border-black-40 md:text-left lg:table-cell lg:w-2/5 lg:pr-32">
                   {item.features && (
@@ -262,7 +262,7 @@ const StudyPlaceFilteringTable = ({items}: Props) => {
                   )}
                 </Td>
                 <Td className="block w-auto sm:border-b sm:border-black-40 md:text-left lg:table-cell lg:w-1/5">
-                  {item.libCalId ? (
+                  {item.libCalId && (
                     <a
                       href={`https://appointments.library.stanford.edu/space/${item.libCalId}`}
                       className="button mb-16 w-fit whitespace-nowrap border border-solid border-cardinal-red bg-white py-[4px] text-16 leading-[22px] text-cardinal-red hocus:bg-cardinal-red hocus:text-white hocus:shadow-button md:w-full lg:mb-0 lg:w-fit"
@@ -274,9 +274,9 @@ const StudyPlaceFilteringTable = ({items}: Props) => {
                         </div>
                       </div>
                     </a>
-                  ) : (
-                    ""
                   )}
+                  {/* Without this, the responsive table library injects a "&nbsp;". */}
+                  {""}
                 </Td>
               </Tr>
             ))}