From 968f8b5df2be7a9fdf2931b358d8d76b47311ec7 Mon Sep 17 00:00:00 2001
From: Mikael Brevik <mikaelbre@gmail.com>
Date: Fri, 29 Nov 2024 20:28:26 +0100
Subject: [PATCH] refactor: better naming on flag

---
 src/components/sections/employees/EmployeeList.tsx | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/components/sections/employees/EmployeeList.tsx b/src/components/sections/employees/EmployeeList.tsx
index a5029ebdc..cd4842b3b 100644
--- a/src/components/sections/employees/EmployeeList.tsx
+++ b/src/components/sections/employees/EmployeeList.tsx
@@ -46,7 +46,7 @@ export default function EmployeeList({
     useState<ChewbaccaEmployee[]>(employees);
 
   const {
-    isShowMore,
+    showShowMoreButton,
     isShowMorePending,
     limitedEmployees,
     showMoreHandler,
@@ -172,7 +172,7 @@ export default function EmployeeList({
         </div>
         {isShowMorePending && <EmployeeListSkeleton />}
 
-        {(!isShowMore || isShowMorePending) && (
+        {showShowMoreButton && (
           <ShowMoreButton
             showMoreHandler={showMoreHandler}
             showMoreHref={showMoreHref}
@@ -235,7 +235,8 @@ function useShowAll(filteredEmployees: ChewbaccaEmployee[]) {
     limitedEmployees,
     showMoreHandler,
     isShowMorePending: isPending,
-    isShowMore: !limitEmployees,
+    showShowMoreButton:
+      limitEmployees && !isPending && filteredEmployees.length > DEFAULT_LIMIT,
     showMoreHref: `${currentPath}?showAll`,
   };
 }