From 0af7b172a014b0c54f22e32a6fbc2f9b70de92b3 Mon Sep 17 00:00:00 2001
From: rstubryan <restubumirr@gmail.com>
Date: Wed, 25 Feb 2026 12:08:31 +0700
Subject: [PATCH] refactor(FE): Refactor PurchaseTable to use Popover for row
 options menu

---
 .../pages/purchase/PurchaseTable.tsx          | 252 ++++++++----------
 1 file changed, 115 insertions(+), 137 deletions(-)

diff --git a/src/components/pages/purchase/PurchaseTable.tsx b/src/components/pages/purchase/PurchaseTable.tsx
index 733165f8..87992ad2 100644
--- a/src/components/pages/purchase/PurchaseTable.tsx
+++ b/src/components/pages/purchase/PurchaseTable.tsx
@@ -12,10 +12,9 @@ import DebouncedTextInput from '@/components/input/DebouncedTextInput';
 import Button from '@/components/Button';
 import { useModal } from '@/components/Modal';
 import ConfirmationModal from '@/components/modal/ConfirmationModal';
+import PopoverButton from '@/components/popover/PopoverButton';
+import PopoverContent from '@/components/popover/PopoverContent';
 import SelectInput, { OptionType } from '@/components/input/SelectInput';
-import RowDropdownOptions from '@/components/table/RowDropdownOptions';
-import RowCollapseOptions from '@/components/table/RowCollapseOptions';
-import RowOptionsMenuWrapper from '@/components/table/RowOptionsMenuWrapper';
 import RequirePermission from '@/components/helper/RequirePermission';
 import StatusBadge from '@/components/helper/StatusBadge';
 
@@ -69,59 +68,72 @@ const getStatusBadgeColor = (status: string): Color => {
   return statusBadgeColorMap[status] || 'neutral';
 };
 
-// ===== INTERFACES =====
-interface RowOptionsMenuProps {
-  type: 'dropdown' | 'collapse';
-  props: CellContext<Purchase, unknown>;
-  deleteClickHandler: () => void;
-}
-
+// ===== ROW OPTIONS MENU =====
 const RowOptionsMenu = ({
-  type = 'dropdown',
+  popoverPosition = 'bottom',
   props,
   deleteClickHandler,
-}: RowOptionsMenuProps) => {
+}: {
+  popoverPosition: 'bottom' | 'top';
+  props: CellContext<Purchase, unknown>;
+  deleteClickHandler: () => void;
+}) => {
+  const popoverId = `purchase#${props.row.original.id}`;
+  const popoverAnchorName = `--anchor-purchase#${props.row.original.id}`;
+
+  const closePopover = () => {
+    document.getElementById(popoverId)?.hidePopover();
+  };
+
   return (
-    <RowOptionsMenuWrapper type={type}>
-      <RequirePermission permissions='lti.purchase.detail'>
-        <Button
-          href={`/purchase/detail/?purchaseId=${props.row.original.id}`}
-          variant='ghost'
-          color='primary'
-          className='justify-start text-sm'
-        >
-          <Icon icon='mdi:eye-outline' width={16} height={16} />
-          Detail
-        </Button>
-      </RequirePermission>
+    <div className='relative'>
+      <PopoverButton
+        tabIndex={0}
+        variant='ghost'
+        color='none'
+        popoverTarget={popoverId}
+        anchorName={popoverAnchorName}
+      >
+        <Icon icon='material-symbols:more-vert' width={16} height={16} />
+      </PopoverButton>
 
-      {/*<Button*/}
-      {/*  href={`/purchase/detail/edit/?purchaseId=${props.row.original.id}`}*/}
-      {/*  variant='ghost'*/}
-      {/*  color='warning'*/}
-      {/*  className='justify-start text-sm'*/}
-      {/*>*/}
-      {/*  <Icon icon='material-symbols:edit-outline' width={16} height={16} />*/}
-      {/*  Edit*/}
-      {/*</Button>*/}
+      <PopoverContent
+        id={popoverId}
+        anchorName={popoverAnchorName}
+        position={popoverPosition === 'bottom' ? 'bottom-start' : 'left'}
+        className='w-full max-w-40 rounded-xl border border-base-content/5 shadow-sm'
+      >
+        <div className='flex flex-col bg-base-100 rounded-xl'>
+          <RequirePermission permissions='lti.purchase.detail'>
+            <Button
+              href={`/purchase/detail/?purchaseId=${props.row.original.id}`}
+              variant='ghost'
+              color='none'
+              className='p-3 justify-start text-sm font-semibold w-full'
+              onClick={closePopover}
+            >
+              <Icon icon='heroicons:eye' width={20} height={20} />
+              Detail
+            </Button>
+          </RequirePermission>
 
-      <RequirePermission permissions='lti.purchase.delete'>
-        <Button
-          onClick={deleteClickHandler}
-          variant='ghost'
-          color='error'
-          className='justify-start text-sm text-error focus-visible:text-error-content hover:text-error-content'
-        >
-          <Icon
-            icon='material-symbols:delete-outline-rounded'
-            width={16}
-            height={16}
-            className='justify-start text-sm'
-          />
-          Delete
-        </Button>
-      </RequirePermission>
-    </RowOptionsMenuWrapper>
+          <RequirePermission permissions='lti.purchase.delete'>
+            <Button
+              onClick={() => {
+                deleteClickHandler();
+                closePopover();
+              }}
+              variant='ghost'
+              color='error'
+              className='p-3 justify-start text-sm font-semibold w-full focus-visible:text-error-content hover:text-error-content'
+            >
+              <Icon icon='mdi:delete-outline' width={20} height={20} />
+              Delete
+            </Button>
+          </RequirePermission>
+        </div>
+      </PopoverContent>
+    </div>
   );
 };
 
@@ -346,27 +358,11 @@ const PurchaseTable = () => {
         };
 
         return (
-          <>
-            {currentPageSize > 2 && (
-              <RowDropdownOptions isLast2Rows={isLast2Rows}>
-                <RowOptionsMenu
-                  type='dropdown'
-                  props={props}
-                  deleteClickHandler={deleteClickHandler}
-                />
-              </RowDropdownOptions>
-            )}
-
-            {currentPageSize <= 2 && (
-              <RowCollapseOptions>
-                <RowOptionsMenu
-                  type='collapse'
-                  props={props}
-                  deleteClickHandler={deleteClickHandler}
-                />
-              </RowCollapseOptions>
-            )}
-          </>
+          <RowOptionsMenu
+            popoverPosition={isLast2Rows ? 'top' : 'bottom'}
+            props={props}
+            deleteClickHandler={deleteClickHandler}
+          />
         );
       },
     },
@@ -405,22 +401,22 @@ const PurchaseTable = () => {
 
   return (
     <>
-      <div className='w-full p-0 sm:p-4'>
-        <div className='flex flex-col gap-2 mb-4'>
-          <div className='w-full flex flex-col xl:flex-row justify-between items-end xl:items-center gap-2'>
-            <div className='w-full flex flex-row gap-2'>
-              <RequirePermission permissions='lti.purchase.create'>
-                <Button
-                  href='/purchase/add'
-                  color='primary'
-                  className='px-3 py-2.5 w-fit text-sm text-base-100 rounded-lg shadow-sm'
-                >
-                  <Icon icon='heroicons:plus' width={20} height={20} />
-                  Add Purchase
-                </Button>
-              </RequirePermission>
-            </div>
+      <div className='w-full'>
+        <div className='w-full p-3 flex flex-row justify-between gap-3 flex-wrap border-b border-base-content/10'>
+          <div className='w-fit flex flex-row gap-3 flex-wrap'>
+            <RequirePermission permissions='lti.purchase.create'>
+              <Button
+                href='/purchase/add'
+                color='primary'
+                className='px-3 py-2.5 w-fit text-sm text-base-100 rounded-xl shadow-button-soft'
+              >
+                <Icon icon='heroicons:plus' width={20} height={20} />
+                Add Purchase
+              </Button>
+            </RequirePermission>
+          </div>
 
+          <div className='flex flex-1 flex-row justify-start sm:justify-end items-center gap-3 flex-wrap'>
             <DebouncedTextInput
               name='search'
               placeholder='Search'
@@ -441,59 +437,41 @@ const PurchaseTable = () => {
               }}
             />
           </div>
-
-          <div className='flex flex-wrap justify-end gap-4'>
-            <SelectInput
-              label='Baris'
-              options={ROWS_OPTIONS}
-              value={{
-                label: String(tableFilterState.pageSize),
-                value: tableFilterState.pageSize,
-              }}
-              onChange={pageSizeChangeHandler}
-              className={{
-                wrapper: 'w-full sm:w-24',
-              }}
-            />
-          </div>
         </div>
 
-        <Table<Purchase>
-          data={
-            isResponseSuccess(purchaseRequests) ? purchaseRequests?.data : []
-          }
-          columns={purchaseColumns}
-          pageSize={tableFilterState.pageSize}
-          page={
-            isResponseSuccess(purchaseRequests)
-              ? purchaseRequests?.meta?.page
-              : 0
-          }
-          totalItems={
-            isResponseSuccess(purchaseRequests)
-              ? purchaseRequests?.meta?.total_results
-              : 0
-          }
-          onPageChange={setPage}
-          isLoading={isLoading}
-          sorting={sorting}
-          setSorting={setSorting}
-          className={{
-            containerClassName: cn({
-              'mb-20':
-                isResponseSuccess(purchaseRequests) &&
-                purchaseRequests?.data?.length === 0,
-            }),
-            tableWrapperClassName: 'overflow-x-auto min-h-full!',
-            tableClassName: 'font-inter w-full table-auto min-h-full!',
-            headerRowClassName: 'border-b border-b-gray-200',
-            headerColumnClassName:
-              'px-6 py-3 text-xs font-semibold text-gray-500 last:flex last:flex-row last:justify-end',
-            bodyRowClassName: 'border-b border-b-gray-200',
-            bodyColumnClassName:
-              'px-6 py-3 last:flex last:flex-row last:justify-end',
-          }}
-        />
+        {/* Table Section */}
+        <div className='flex flex-col mb-4'>
+          <Table<Purchase>
+            data={
+              isResponseSuccess(purchaseRequests) ? purchaseRequests?.data : []
+            }
+            columns={purchaseColumns}
+            pageSize={tableFilterState.pageSize}
+            page={
+              isResponseSuccess(purchaseRequests)
+                ? purchaseRequests?.meta?.page
+                : 0
+            }
+            totalItems={
+              isResponseSuccess(purchaseRequests)
+                ? purchaseRequests?.meta?.total_results
+                : 0
+            }
+            onPageChange={setPage}
+            onPageSizeChange={setPageSize}
+            isLoading={isLoading}
+            sorting={sorting}
+            setSorting={setSorting}
+            className={{
+              containerClassName: cn('p-3', {
+                'w-full mb-20':
+                  isResponseSuccess(purchaseRequests) &&
+                  purchaseRequests?.data?.length === 0,
+              }),
+              headerColumnClassName: 'text-nowrap',
+            }}
+          />
+        </div>
       </div>
 
       {/* ===== MODAL COMPONENTS ===== */}