From 5714ed16d72c5442f58366ac51f40cc32ea406ab Mon Sep 17 00:00:00 2001
From: Som Gupta <78577376+knownotunknown@users.noreply.github.com>
Date: Thu, 14 Mar 2024 00:17:47 -0500
Subject: [PATCH] fix: list component fixed (#162)

* fix: list component fixed

* style: ran prettier

* Update List.tsx

* refactor: remove console log

---------

Co-authored-by: Razboy20 <razboy20@gmail.com>
---
 .../calendar/CalendarHeader/CalenderHeader.tsx           | 4 ++--
 src/views/components/common/List/List.tsx                | 9 ++++-----
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/views/components/calendar/CalendarHeader/CalenderHeader.tsx b/src/views/components/calendar/CalendarHeader/CalenderHeader.tsx
index 149515d3..3bfbe2da 100644
--- a/src/views/components/calendar/CalendarHeader/CalenderHeader.tsx
+++ b/src/views/components/calendar/CalendarHeader/CalenderHeader.tsx
@@ -33,7 +33,7 @@ export default function CalendarHeader(): JSX.Element {
 
     return (
         <div className='min-h-79px min-w-672px w-full flex px-0 py-5'>
-            <div className='flex flex-row gap-20 w-full'>
+            <div className='w-full flex flex-row gap-20'>
                 <div className='flex gap-10'>
                     <div className='flex gap-1'>
                         <Button className='self-center' variant='single' icon={MenuIcon} color='ut-gray' />
@@ -57,7 +57,7 @@ export default function CalendarHeader(): JSX.Element {
                         </Text>
                     </div>
                 </div>
-                <div className='flex flex-row items-center justify-end space-x-8 ml-auto'>
+                <div className='ml-auto flex flex-row items-center justify-end space-x-8'>
                     <div className='flex flex-row space-x-4'>
                         <CourseStatus size='small' status={Status.WAITLISTED} />
                         <CourseStatus size='small' status={Status.CLOSED} />
diff --git a/src/views/components/common/List/List.tsx b/src/views/components/common/List/List.tsx
index 7a056b74..7f5bbb8b 100644
--- a/src/views/components/common/List/List.tsx
+++ b/src/views/components/common/List/List.tsx
@@ -75,13 +75,12 @@ function List<T>(props: ListProps<T>): JSX.Element {
 
     useEffect(() => {
         // check if the draggables content has *actually* changed
-        if (props.draggables.every((element, index) => equalityCheck(element, items[index].content))) {
-            console.log("List's draggables have not changed");
+        if (
+            props.draggables.length === items.length &&
+            props.draggables.every((element, index) => equalityCheck(element, items[index].content))
+        ) {
             return;
         }
-
-        console.log("List's draggables have changed, updating...");
-
         setItems(wrap(props.draggables));
     }, [props.draggables]);