Merge pull request #1458 from janvi01/bulkdeletionsConfirmation

Added confirmation for bulk deletion of rows
rowyio · Nov 8, 2023 · 46826bc · 46826bc
2 parents b64bdbc + bb552a5
commit 46826bc
Showing 1 changed file with 18 additions and 1 deletion.
diff --git a/src/components/TableToolbar/TableToolbar.tsx b/src/components/TableToolbar/TableToolbar.tsx
@@ -26,6 +26,8 @@ import {
   userRolesAtom,
   compatibleRowyRunVersionAtom,
   rowyRunModalAtom,
+  altPressAtom,
+  confirmDialogAtom,
 } from "@src/atoms/projectScope";
 import {
   tableScope,
@@ -91,6 +93,8 @@ function RowSelectedToolBar({
 }) {
   const [serverDocCount] = useAtom(serverDocCountAtom, tableScope);
   const deleteRow = useSetAtom(deleteRowAtom, tableScope);
+  const [altPress] = useAtom(altPressAtom, projectScope);
+  const confirm = useSetAtom(confirmDialogAtom, projectScope);
 
   const handleDelete = async () => {
     await deleteRow({ path: Object.keys(selectedRows) });
@@ -107,7 +111,20 @@ function RowSelectedToolBar({
           variant="outlined"
           startIcon={<DeleteIcon fontSize="small" />}
           color="error"
-          onClick={handleDelete}
+          onClick={
+            altPress
+              ? handleDelete
+              : () => {
+                  confirm({
+                    title: `Delete ${
+                      Object.values(selectedRows).length
+                    } of ${serverDocCount} selected rows?`,
+                    confirm: "Delete",
+                    confirmColor: "error",
+                    handleConfirm: handleDelete,
+                  });
+                }
+          }
         >
           Delete
         </Button>