From d193d9bf89e8a88bf70b673ea5e438d73cf40c0c Mon Sep 17 00:00:00 2001
From: Md Saban <45597394+mdsaban@users.noreply.github.com>
Date: Tue, 2 Jul 2024 03:51:23 +0530
Subject: feat: Add bulk edit option for bookmarks. Fixes #84 (#259)

* feat: add bulk edit option for bookmarks

* fix: resolve comments

* fix: resolve comments

* fix: resolve comments

* fix: resolve comments

* rename bulk action store, simplify the bulk action toolbar

---------

Co-authored-by: MohamedBassem <me@mbassem.com>
---
 apps/web/components/ui/action-button.tsx            | 2 +-
 apps/web/components/ui/action-confirming-dialog.tsx | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

(limited to 'apps/web/components/ui')

diff --git a/apps/web/components/ui/action-button.tsx b/apps/web/components/ui/action-button.tsx
index b3984d97..b7cd9b3d 100644
--- a/apps/web/components/ui/action-button.tsx
+++ b/apps/web/components/ui/action-button.tsx
@@ -46,7 +46,7 @@ const ActionButtonWithTooltip = React.forwardRef<
 >(({ tooltip, delayDuration, ...props }, ref) => {
   return (
     <Tooltip delayDuration={delayDuration}>
-      <TooltipTrigger>
+      <TooltipTrigger asChild>
         <ActionButton ref={ref} {...props} />
       </TooltipTrigger>
       <TooltipPortal>
diff --git a/apps/web/components/ui/action-confirming-dialog.tsx b/apps/web/components/ui/action-confirming-dialog.tsx
index 37895ee7..cfd38fc3 100644
--- a/apps/web/components/ui/action-confirming-dialog.tsx
+++ b/apps/web/components/ui/action-confirming-dialog.tsx
@@ -24,7 +24,7 @@ export default function ActionConfirmingDialog({
   title: React.ReactNode;
   description: React.ReactNode;
   actionButton: (setDialogOpen: (open: boolean) => void) => React.ReactNode;
-  children: React.ReactNode;
+  children?: React.ReactNode;
 }) {
   const [customIsOpen, setCustomIsOpen] = useState(false);
   const [isDialogOpen, setDialogOpen] = [
@@ -33,7 +33,7 @@ export default function ActionConfirmingDialog({
   ];
   return (
     <Dialog open={isDialogOpen} onOpenChange={setDialogOpen}>
-      <DialogTrigger asChild>{children}</DialogTrigger>
+      {children && <DialogTrigger asChild>{children}</DialogTrigger>}
       <DialogContent>
         <DialogHeader>
           <DialogTitle>{title}</DialogTitle>
-- 
cgit v1.2.3-70-g09d2