From 43563bc8d59181ec2b93b5fc29ff601f3b0b14f6 Mon Sep 17 00:00:00 2001
From: Peer Richelsen <peeroke@gmail.com>
Date: Wed, 22 Sep 2021 15:44:38 +0100
Subject: [PATCH] made it easier to see if there are more times available to
 scroll (#722)

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
---
 components/booking/AvailableTimes.tsx | 90 ++++++++++++++-------------
 1 file changed, 46 insertions(+), 44 deletions(-)

diff --git a/components/booking/AvailableTimes.tsx b/components/booking/AvailableTimes.tsx
index 83266bad74..887843315b 100644
--- a/components/booking/AvailableTimes.tsx
+++ b/components/booking/AvailableTimes.tsx
@@ -29,63 +29,65 @@ const AvailableTimes = ({
   });
 
   return (
-    <div className="sm:pl-4 mt-8 sm:mt-0 text-center sm:w-1/3 md:max-h-97 overflow-y-auto">
+    <div className="sm:pl-4 mt-8 sm:mt-0 text-center sm:w-1/3 -mb-5">
       <div className="text-gray-600 font-light text-lg mb-4 text-left">
         <span className="w-1/2 dark:text-white text-gray-600">
           <strong>{date.format("dddd")}</strong>
           <span className="text-gray-500">{date.format(", DD MMMM")}</span>
         </span>
       </div>
-      {!loading &&
-        slots?.length > 0 &&
-        slots.map((slot) => {
-          const bookingUrl = {
-            pathname: "book",
-            query: {
-              ...router.query,
-              date: slot.time.format(),
-              type: eventTypeId,
-            },
-          };
+      <div className="md:max-h-[364px] overflow-y-auto">
+        {!loading &&
+          slots?.length > 0 &&
+          slots.map((slot) => {
+            const bookingUrl = {
+              pathname: "book",
+              query: {
+                ...router.query,
+                date: slot.time.format(),
+                type: eventTypeId,
+              },
+            };
 
-          if (rescheduleUid) {
-            bookingUrl.query.rescheduleUid = rescheduleUid;
-          }
+            if (rescheduleUid) {
+              bookingUrl.query.rescheduleUid = rescheduleUid;
+            }
 
-          if (schedulingType === SchedulingType.ROUND_ROBIN) {
-            bookingUrl.query.user = slot.users;
-          }
+            if (schedulingType === SchedulingType.ROUND_ROBIN) {
+              bookingUrl.query.user = slot.users;
+            }
 
-          return (
-            <div key={slot.time.format()}>
-              <Link href={bookingUrl}>
-                <a className="block font-medium mb-4 bg-white dark:bg-gray-600 text-primary-500 dark:text-neutral-200 border border-primary-500 dark:border-transparent rounded-sm hover:text-white hover:bg-primary-500 dark:hover:border-black py-4 dark:hover:bg-black">
-                  {slot.time.format(timeFormat)}
-                </a>
-              </Link>
-            </div>
-          );
-        })}
-      {!loading && !error && !slots.length && (
-        <div className="w-full h-full flex flex-col justify-center content-center items-center -mt-4">
-          <h1 className="text-xl text-black dark:text-white">All booked today.</h1>
-        </div>
-      )}
+            return (
+              <div key={slot.time.format()}>
+                <Link href={bookingUrl}>
+                  <a className="block font-medium mb-2 bg-white dark:bg-gray-600 text-primary-500 dark:text-neutral-200 border border-primary-500 dark:border-transparent rounded-sm hover:text-white hover:bg-primary-500 dark:hover:border-black py-4 dark:hover:bg-black">
+                    {slot.time.format(timeFormat)}
+                  </a>
+                </Link>
+              </div>
+            );
+          })}
+        {!loading && !error && !slots.length && (
+          <div className="w-full h-full flex flex-col justify-center content-center items-center -mt-4">
+            <h1 className="text-xl text-black dark:text-white">All booked today.</h1>
+          </div>
+        )}
 
-      {loading && <Loader />}
+        {loading && <Loader />}
 
-      {error && (
-        <div className="bg-yellow-50 border-l-4 border-yellow-400 p-4">
-          <div className="flex">
-            <div className="flex-shrink-0">
-              <ExclamationIcon className="h-5 w-5 text-yellow-400" aria-hidden="true" />
-            </div>
-            <div className="ml-3">
-              <p className="text-sm text-yellow-700">Could not load the available time slots.</p>
+        {error && (
+          <div className="bg-yellow-50 border-l-4 border-yellow-400 p-4">
+            <div className="flex">
+              <div className="flex-shrink-0">
+                <ExclamationIcon className="h-5 w-5 text-yellow-400" aria-hidden="true" />
+              </div>
+              <div className="ml-3">
+                <p className="text-sm text-yellow-700">Could not load the available time slots.</p>
+              </div>
             </div>
           </div>
-        </div>
-      )}
+        )}
+      </div>
     </div>
   );
 };