refactor: better naming on flag

varianter · Dec 2, 2024 · a098d7a · a098d7a
1 parent d6bee61
commit a098d7a
Showing 1 changed file with 4 additions and 3 deletions.
diff --git 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`,
   };
 }