From e536a52815a9f19f2392b8f61be12f97cfd8cc3a Mon Sep 17 00:00:00 2001
From: Matthew Warman <warmanmr@gmail.com>
Date: Tue, 4 Mar 2025 13:41:45 -0500
Subject: [PATCH 1/5] #57 Dialog compound pattern

---
 src/common/components/Dialog/Dialog.tsx       | 290 +++++++++++++++---
 src/common/components/Dialog/DialogButton.tsx |  53 ----
 .../components/Dialog/DialogButtons.tsx       |  32 --
 .../components/Dialog/DialogContent.tsx       |  31 --
 .../components/Dialog/DialogHeading.tsx       |  31 --
 .../Dialog/__tests__/DiaglogButtons.test.tsx  |  47 ---
 .../Dialog/__tests__/Dialog.test.tsx          | 224 +++++++++++---
 .../Dialog/__tests__/DialogButton.test.tsx    |  53 ----
 .../Dialog/__tests__/DialogContent.test.tsx   |  34 --
 .../Dialog/__tests__/DialogHeading.test.tsx   |  34 --
 .../components/Delete/TaskDeleteDialog.tsx    | 116 ++++---
 .../__tests__/TaskDeleteDialog.test.tsx       |  78 ++---
 .../Tasks/components/TaskDetailLayout.tsx     |  75 +----
 .../__tests__/TaskDetailLayout.test.tsx       |  81 +----
 14 files changed, 569 insertions(+), 610 deletions(-)
 delete mode 100644 src/common/components/Dialog/DialogButton.tsx
 delete mode 100644 src/common/components/Dialog/DialogButtons.tsx
 delete mode 100644 src/common/components/Dialog/DialogContent.tsx
 delete mode 100644 src/common/components/Dialog/DialogHeading.tsx
 delete mode 100644 src/common/components/Dialog/__tests__/DiaglogButtons.test.tsx
 delete mode 100644 src/common/components/Dialog/__tests__/DialogButton.test.tsx
 delete mode 100644 src/common/components/Dialog/__tests__/DialogContent.test.tsx
 delete mode 100644 src/common/components/Dialog/__tests__/DialogHeading.test.tsx

diff --git a/src/common/components/Dialog/Dialog.tsx b/src/common/components/Dialog/Dialog.tsx
index a1e7b7c..60afd84 100644
--- a/src/common/components/Dialog/Dialog.tsx
+++ b/src/common/components/Dialog/Dialog.tsx
@@ -1,63 +1,114 @@
-import { PropsWithChildren, useEffect, useState } from 'react';
+import { createContext, PropsWithChildren, ReactNode, useContext, useState } from 'react';
+import noop from 'lodash/noop';
+import { cva, VariantProps } from 'class-variance-authority';
 
-import { cn } from 'common/utils/css';
 import { BaseComponentProps } from 'common/utils/types';
+import { cn } from 'common/utils/css';
 import Backdrop from '../Backdrop/Backdrop';
+import Divider, { DividerProps } from '../Divider/Divider';
+import { default as CommonButton, ButtonProps as CommonButtonProps } from '../Button/Button';
+
+/**
+ * Defines the properties of the `DialogContext` value.
+ */
+type DialogContextValue = {
+  isHidden: boolean;
+  setIsHidden: (isHidden: boolean) => void;
+};
+
+/**
+ * The `DialogContext` instance.
+ */
+const DialogContext = createContext<DialogContextValue>({
+  isHidden: true,
+  setIsHidden: noop,
+});
 
 /**
- * Properties for the `Dialog` component.
- * @param {boolean} [isOpen] - Indicates if the Dialog should be displayed.
- * @param {function} [onClose] - A function called when the Dialog closes.
- * @see {@link BaseComponentProps}
- * @see {@link PropsWithChildren}
+ * Defines the properties of the `Dialog` render prop function context.
+ * @param close - Close the dialog.
  */
-export interface DialogProps extends BaseComponentProps, PropsWithChildren {
-  isOpen?: boolean;
-  onClose?: () => void | Promise<void>;
+type DialogRenderFnContext = {
+  close: () => void;
+};
+
+/**
+ * The `Dialog` render prop function signature.
+ */
+type DialogRenderFn = (ctx: DialogRenderFnContext) => ReactNode;
+
+export interface DialogProps extends BaseComponentProps {
+  children: ReactNode | DialogRenderFn;
 }
 
 /**
  * A `Dialog` is a modal window that displays on top of the main content,
  * typically asking the user to take an action or confirm a decision.
- * @param {DialogProps} props - Component properties.
- * @returns {JSX.Element} JSX
  */
-const Dialog = ({
+const Dialog = ({ children, className, testId = 'dialog' }: DialogProps): JSX.Element => {
+  const [isHidden, setIsHidden] = useState(true);
+
+  const close = () => {
+    setIsHidden(true);
+  };
+
+  return (
+    <div className={cn(className)} data-testid={testId}>
+      <DialogContext.Provider value={{ isHidden, setIsHidden }}>
+        {typeof children === 'function' ? children({ close }) : children}
+      </DialogContext.Provider>
+    </div>
+  );
+};
+
+/**
+ * The `Trigger` renders a clickable element used to open a `Dialog`. There
+ * should be 1 `Trigger` within a `Dialog`.
+ */
+const Trigger = ({
   children,
   className,
-  isOpen = false,
-  onClose,
-  testId = 'dialog',
-}: DialogProps): JSX.Element => {
-  const [isDialogOpen, setIsDialogOpen] = useState(isOpen);
-
-  useEffect(() => {
-    setIsDialogOpen(isOpen);
-  }, [isOpen]);
-
-  const closeDialog = (): void => {
-    setIsDialogOpen(false);
-    onClose?.();
-  };
+  testId = 'dialog-trigger',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  const { isHidden, setIsHidden } = useContext(DialogContext);
 
-  const handleBackdropClick = (): void => {
-    closeDialog();
-  };
+  return (
+    <div
+      role="button"
+      className={cn('cursor-pointer hover:opacity-80', className)}
+      onClick={() => setIsHidden(!isHidden)}
+      data-testid={testId}
+    >
+      {children}
+    </div>
+  );
+};
+Dialog.Trigger = Trigger;
 
-  const handleDialogClick = (e: React.MouseEvent): void => {
-    e.stopPropagation();
-  };
+/**
+ * The `Content` component wraps the contents of a `Dialog` including the
+ * header, body, and footer. There should be 1 `Content` within a `Dialog`.
+ */
+const Content = ({
+  children,
+  className,
+  testId = 'dialog-content',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  const { isHidden, setIsHidden } = useContext(DialogContext);
 
   return (
-    <div className={cn({ hidden: !isDialogOpen }, className)} data-testid={testId}>
+    <div className={cn({ hidden: isHidden }, className)} data-testid={testId}>
       <Backdrop
         className="flex items-center justify-center"
-        onClick={handleBackdropClick}
+        onClick={() => setIsHidden(true)}
         testId={`${testId}-backdrop`}
       >
         <div
-          className="m-4 min-w-72 max-w-[560px] rounded-3xl bg-light-bg p-6 dark:bg-dark-bg"
-          onClick={handleDialogClick}
+          className={cn(
+            'bg-light-bg dark:bg-dark-bg m-4 flex max-w-[560px] min-w-72 flex-col gap-4 rounded-md p-6',
+          )}
+          onClick={(e) => e.stopPropagation()}
+          data-testid={`${testId}-content`}
         >
           {children}
         </div>
@@ -65,5 +116,170 @@ const Dialog = ({
     </div>
   );
 };
+Dialog.Content = Content;
+
+/**
+ * The `Header` is a block within a dialog `Content` and may contain a `Title`
+ * and `Subtitle`.
+ */
+const Header = ({
+  children,
+  className,
+  testId = 'dialog-header',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  return (
+    <div className={cn(className)} data-testid={testId}>
+      {children}
+    </div>
+  );
+};
+Dialog.Header = Header;
+
+/**
+ * The `Body` is a block which encloses the main content of the `Dialog`.
+ */
+const Body = ({
+  children,
+  className,
+  testId = 'dialog-body',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  return (
+    <div className={cn(className)} data-testid={testId}>
+      {children}
+    </div>
+  );
+};
+Dialog.Body = Body;
+
+/**
+ * The `Footer` is a block within a dialog `Content` and contains values such as
+ * a `ButtonBar`, `Button`, or any components which are located at the bottom of
+ * the `Dialog`.
+ */
+const Footer = ({
+  children,
+  className,
+  testId = 'dialog-footer',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  return (
+    <div className={cn(className)} data-testid={testId}>
+      {children}
+    </div>
+  );
+};
+Dialog.Footer = Footer;
+
+/**
+ * A `Title` for the `Dialog`.  The title is optional.  When present, the `Title`
+ * is typically located within the dialog `Header`.
+ */
+const Title = ({
+  children,
+  className,
+  testId = 'dialog-title',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  return (
+    <h5 className={cn('line-clamp-2 text-2xl', className)} data-testid={testId}>
+      {children}
+    </h5>
+  );
+};
+Dialog.Title = Title;
+
+/**
+ * A `Subtitle` for the `Dialog`. The subtitle is optional. When present, the
+ * `Subtitle` is typically located within the dialog `Header`, immediately after
+ * the `Title`.
+ */
+const Subtitle = ({
+  children,
+  className,
+  testId = 'dialog-subtitle',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  return (
+    <div
+      className={cn(
+        'line-clamp-2 leading-tight text-neutral-500 font-stretch-condensed',
+        className,
+      )}
+      data-testid={testId}
+    >
+      {children}
+    </div>
+  );
+};
+Dialog.Subtitle = Subtitle;
+
+/**
+ * A `ButtonBar` organizes one to many dialog `Button` components in a horizontal
+ * row.  The buttons are right justified within the bar.
+ */
+const ButtonBar = ({
+  children,
+  className,
+  testId = 'dialog-button-bar',
+}: BaseComponentProps & PropsWithChildren): JSX.Element => {
+  return (
+    <div className={cn('flex items-center justify-end gap-4', className)} data-testid={testId}>
+      {children}
+    </div>
+  );
+};
+Dialog.ButtonBar = ButtonBar;
+
+/**
+ * Define the `Button` component base and variant styles.
+ */
+const buttonVariants = cva('', {
+  variants: {
+    variant: {
+      danger: 'font-bold text-red-600',
+      primary: 'font-bold text-blue-600 dark:text-blue-400',
+      secondary: '',
+    },
+  },
+  defaultVariants: { variant: 'secondary' },
+});
+
+/**
+ * The variant attributes of the `Button` component.
+ */
+type ButtonVariants = VariantProps<typeof buttonVariants>;
+
+/**
+ * Properties for the `Button` component.
+ */
+export interface ButtonProps extends Omit<CommonButtonProps, 'variant'>, ButtonVariants {}
+
+/**
+ * A dialog `Button` is a button which is styles for presentation within a
+ * `Dialog`.
+ */
+const Button = ({
+  className,
+  variant = 'secondary',
+  testId = 'dialog-footer',
+  ...props
+}: ButtonProps): JSX.Element => {
+  return (
+    <CommonButton
+      variant="text"
+      size="sm"
+      className={cn(buttonVariants({ variant, className }))}
+      testId={testId}
+      {...props}
+    />
+  );
+};
+Dialog.Button = Button;
+
+/**
+ * The `Separator` component renders a horizontal divider.
+ * This is useful to organize and separate content.
+ */
+const Separator = ({ className, testId = 'dialog-separator' }: DividerProps): JSX.Element => {
+  return <Divider className={cn('my-1', className)} testId={testId} />;
+};
+Dialog.Separator = Separator;
 
 export default Dialog;
diff --git a/src/common/components/Dialog/DialogButton.tsx b/src/common/components/Dialog/DialogButton.tsx
deleted file mode 100644
index 953661c..0000000
--- a/src/common/components/Dialog/DialogButton.tsx
+++ /dev/null
@@ -1,53 +0,0 @@
-import { cva, VariantProps } from 'class-variance-authority';
-
-import { cn } from 'common/utils/css';
-import Button, { ButtonProps } from '../Button/Button';
-
-/**
- * Define the component base and variant styles.
- */
-const variants = cva('', {
-  variants: {
-    variant: {
-      danger: 'font-bold text-red-600',
-      primary: 'font-bold text-blue-600 dark:text-blue-400',
-      secondary: '',
-    },
-  },
-  defaultVariants: { variant: 'secondary' },
-});
-
-/**
- * The variant attributes of the DialogButton component.
- */
-type DialogButtonVariants = VariantProps<typeof variants>;
-
-/**
- * Properties for the `DialogButton` component.
- */
-export interface DialogButtonProps extends Omit<ButtonProps, 'variant'>, DialogButtonVariants {}
-
-/**
- * The `DialogButton` is a type of `Button` specifically styled for use
- * within a `Dialog`.
- * @param {DialogButtonProps} props - Component properties.
- * @returns {JSX.Element} JSX
- */
-const DialogButton = ({
-  className,
-  variant,
-  testId = 'dialog-button',
-  ...buttonProps
-}: DialogButtonProps): JSX.Element => {
-  return (
-    <Button
-      variant="text"
-      size="sm"
-      className={cn(variants({ variant, className }))}
-      testId={testId}
-      {...buttonProps}
-    />
-  );
-};
-
-export default DialogButton;
diff --git a/src/common/components/Dialog/DialogButtons.tsx b/src/common/components/Dialog/DialogButtons.tsx
deleted file mode 100644
index dc724bd..0000000
--- a/src/common/components/Dialog/DialogButtons.tsx
+++ /dev/null
@@ -1,32 +0,0 @@
-import { PropsWithChildren } from 'react';
-
-import { BaseComponentProps } from 'common/utils/types';
-import { cn } from 'common/utils/css';
-
-/**
- * Properties for the `DialogButtons` component.
- * @see {@link BaseComponentProps}
- * @see {@link PropsWithChildren}
- */
-export interface DialogButtonsProps extends BaseComponentProps, PropsWithChildren {}
-
-/**
- * The `DialogButtons` component is a container for one or more `DialogButton`
- * components.  `DialogButtons` renders the buttons, the `children`, styled
- * appropriately for appearing within a `Dialog`.
- * @param {DialogButtonsProps} props - Component properties
- * @returns {JSX.Element} JSX
- */
-const DialogButtons = ({
-  children,
-  className,
-  testId = 'dialog-buttons',
-}: DialogButtonsProps): JSX.Element => {
-  return (
-    <div className={cn('mt-6 flex items-center justify-end gap-4', className)} data-testid={testId}>
-      {children}
-    </div>
-  );
-};
-
-export default DialogButtons;
diff --git a/src/common/components/Dialog/DialogContent.tsx b/src/common/components/Dialog/DialogContent.tsx
deleted file mode 100644
index 2e80259..0000000
--- a/src/common/components/Dialog/DialogContent.tsx
+++ /dev/null
@@ -1,31 +0,0 @@
-import { PropsWithChildren } from 'react';
-
-import { BaseComponentProps } from 'common/utils/types';
-import { cn } from 'common/utils/css';
-
-/**
- * Properties for the `DialogContent` component.
- * @see {@link BaseComponentProps}
- * @see {@link PropsWithChildren}
- */
-export interface DialogContentProps extends BaseComponentProps, PropsWithChildren {}
-
-/**
- * The `DialogContent` component serves as the container for the main content
- * of a `Dialog`. All of the main body content should be wrapped by `DialogContent`.
- * @param {DialogContentProps} props - Component properties.
- * @returns {JSX.Element} JSX
- */
-const DialogContent = ({
-  children,
-  className,
-  testId = 'dialog-content',
-}: DialogContentProps): JSX.Element => {
-  return (
-    <div className={cn('mb-6 mt-4 text-sm', className)} data-testid={testId}>
-      {children}
-    </div>
-  );
-};
-
-export default DialogContent;
diff --git a/src/common/components/Dialog/DialogHeading.tsx b/src/common/components/Dialog/DialogHeading.tsx
deleted file mode 100644
index df95f67..0000000
--- a/src/common/components/Dialog/DialogHeading.tsx
+++ /dev/null
@@ -1,31 +0,0 @@
-import { PropsWithChildren } from 'react';
-
-import { BaseComponentProps } from 'common/utils/types';
-import { cn } from 'common/utils/css';
-
-/**
- * Properties for the `DialogHeading` component.
- * @see {@link BaseComponentProps}
- * @see {@link PropsWithChildren}
- */
-export interface DialogHeadingProps extends BaseComponentProps, PropsWithChildren {}
-
-/**
- * The `DialogHeading` component serves as the container for the heading content
- * of a `Dialog`. All of the heading content should be wrapped by `DialogHeading`.
- * @param {DialogHeadingProps} props - Component properties.
- * @returns {JSX.Element} JSX
- */
-const DialogHeading = ({
-  children,
-  className,
-  testId = 'dialog-heading',
-}: DialogHeadingProps): JSX.Element => {
-  return (
-    <div className={cn('mb-4 line-clamp-2 text-2xl', className)} data-testid={testId}>
-      {children}
-    </div>
-  );
-};
-
-export default DialogHeading;
diff --git a/src/common/components/Dialog/__tests__/DiaglogButtons.test.tsx b/src/common/components/Dialog/__tests__/DiaglogButtons.test.tsx
deleted file mode 100644
index 0df883c..0000000
--- a/src/common/components/Dialog/__tests__/DiaglogButtons.test.tsx
+++ /dev/null
@@ -1,47 +0,0 @@
-import { describe, expect, it } from 'vitest';
-
-import { render, screen } from 'test/test-utils';
-
-import DialogButtons from '../DialogButtons';
-
-describe('DialogButtons', () => {
-  it('should render successfully', async () => {
-    // ARRANGE
-    render(
-      <DialogButtons>
-        <div data-testid="children"></div>
-      </DialogButtons>,
-    );
-    await screen.findByTestId('dialog-buttons');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-buttons')).toBeDefined();
-    expect(screen.getByTestId('children')).toBeDefined();
-  });
-
-  it('should use testId', async () => {
-    // ARRANGE
-    render(
-      <DialogButtons testId="my-dialog-buttons">
-        <div data-testid="children"></div>
-      </DialogButtons>,
-    );
-    await screen.findByTestId('my-dialog-buttons');
-
-    // ASSERT
-    expect(screen.getByTestId('my-dialog-buttons')).toBeDefined();
-  });
-
-  it('should use className', async () => {
-    // ARRANGE
-    render(
-      <DialogButtons className="my-class">
-        <div data-testid="children"></div>
-      </DialogButtons>,
-    );
-    await screen.findByTestId('dialog-buttons');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-buttons')).toHaveClass('my-class');
-  });
-});
diff --git a/src/common/components/Dialog/__tests__/Dialog.test.tsx b/src/common/components/Dialog/__tests__/Dialog.test.tsx
index accb1f9..f981d7f 100644
--- a/src/common/components/Dialog/__tests__/Dialog.test.tsx
+++ b/src/common/components/Dialog/__tests__/Dialog.test.tsx
@@ -1,5 +1,5 @@
 import userEvent from '@testing-library/user-event';
-import { describe, expect, it, vi } from 'vitest';
+import { describe, expect, it } from 'vitest';
 
 import { render, screen, waitFor } from 'test/test-utils';
 
@@ -8,91 +8,233 @@ import Dialog from '../Dialog';
 describe('Dialog', () => {
   it('should render successfully', async () => {
     // ARRANGE
-    render(<Dialog />);
+    render(
+      <Dialog>
+        {({ close }) => (
+          <>
+            <Dialog.Trigger>Open</Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Title</Dialog.Title>
+                <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>Body</Dialog.Body>
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </>
+        )}
+      </Dialog>,
+    );
     await screen.findByTestId('dialog');
 
     // ASSERT
     expect(screen.getByTestId('dialog')).toBeDefined();
   });
 
-  it('should use testId', async () => {
-    // ARRANGE
-    render(<Dialog testId="my-dialog" />);
-    await screen.findByTestId('my-dialog');
-
-    // ASSERT
-    expect(screen.getByTestId('my-dialog')).toBeDefined();
-  });
-
-  it('should use className', async () => {
+  it('should render successfully with ReactNode children', async () => {
     // ARRANGE
-    render(<Dialog className="my-class" />);
+    render(
+      <Dialog>
+        <Dialog.Trigger>Open</Dialog.Trigger>
+        <Dialog.Content>
+          <Dialog.Header>
+            <Dialog.Title>Title</Dialog.Title>
+            <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+          </Dialog.Header>
+          <Dialog.Body>Body</Dialog.Body>
+          <Dialog.Footer>
+            <Dialog.ButtonBar>
+              <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+            </Dialog.ButtonBar>
+          </Dialog.Footer>
+        </Dialog.Content>
+      </Dialog>,
+    );
     await screen.findByTestId('dialog');
 
     // ASSERT
-    expect(screen.getByTestId('dialog')).toHaveClass('my-class');
+    expect(screen.getByTestId('dialog')).toBeDefined();
   });
-
   it('should be hidden by default', async () => {
     // ARRANGE
-    render(<Dialog />);
-    await screen.findByTestId('dialog');
+    render(
+      <Dialog>
+        {({ close }) => (
+          <>
+            <Dialog.Trigger>Open</Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Title</Dialog.Title>
+                <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>Body</Dialog.Body>
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </>
+        )}
+      </Dialog>,
+    );
+    await screen.findByTestId('dialog-content');
 
     // ASSERT
-    expect(screen.getByTestId('dialog')).toHaveClass('hidden');
+    expect(screen.getByTestId('dialog-content')).toHaveClass('hidden');
   });
 
-  it('should not be hidden when isOpen true', async () => {
+  it('should open when trigger is clicked', async () => {
     // ARRANGE
-    render(<Dialog isOpen={true} />);
-    await screen.findByTestId('dialog');
+    const user = userEvent.setup();
+    render(
+      <Dialog>
+        {({ close }) => (
+          <>
+            <Dialog.Trigger>Open</Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Title</Dialog.Title>
+                <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>Body</Dialog.Body>
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </>
+        )}
+      </Dialog>,
+    );
+    await screen.findByTestId('dialog-trigger');
+
+    // ACT
+    await user.click(screen.getByTestId('dialog-trigger'));
 
     // ASSERT
-    expect(screen.getByTestId('dialog')).not.toHaveClass('hidden');
+    expect(screen.getByTestId('dialog-content')).not.toHaveClass('hidden');
   });
 
-  it('should be hidden when isOpen false', async () => {
+  it('should close when cancel button is clicked', async () => {
     // ARRANGE
-    render(<Dialog isOpen={false} />);
-    await screen.findByTestId('dialog');
+    const user = userEvent.setup();
+    render(
+      <Dialog>
+        {({ close }) => (
+          <>
+            <Dialog.Trigger>Open</Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Title</Dialog.Title>
+                <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>Body</Dialog.Body>
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button onClick={() => close()} testId="dialog-button-cancel">
+                    Cancel
+                  </Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </>
+        )}
+      </Dialog>,
+    );
+    await screen.findByTestId('dialog-trigger');
+
+    // ACT
+    await user.click(screen.getByTestId('dialog-trigger'));
+    /* dialog is open */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).not.toHaveClass('hidden'));
+
+    await user.click(screen.getByTestId('dialog-button-cancel'));
+    /* dialog is closed */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).toHaveClass('hidden'));
 
     // ASSERT
-    expect(screen.getByTestId('dialog')).toHaveClass('hidden');
+    expect(screen.getByTestId('dialog-content')).toHaveClass('hidden');
   });
 
   it('should close when backdrop clicked', async () => {
     // ARRANGE
     const user = userEvent.setup();
-    const closeFn = vi.fn();
-    render(<Dialog onClose={closeFn} isOpen={true} />);
-    await screen.findByTestId('dialog');
-    waitFor(() => expect(screen.getByTestId('dialog')).not.toHaveClass('hidden'));
+    render(
+      <Dialog>
+        {({ close }) => (
+          <>
+            <Dialog.Trigger>Open</Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Title</Dialog.Title>
+                <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>Body</Dialog.Body>
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </>
+        )}
+      </Dialog>,
+    );
+    await screen.findByTestId('dialog-trigger');
 
     // ACT
-    await user.click(screen.getByTestId('dialog-backdrop'));
+    await user.click(screen.getByTestId('dialog-trigger'));
+    /* dialog is open */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).not.toHaveClass('hidden'));
+
+    await user.click(screen.getByTestId('dialog-content-backdrop'));
+    /* dialog is closed */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).toHaveClass('hidden'));
 
     // ASSERT
-    expect(closeFn).toHaveBeenCalledOnce();
-    expect(screen.getByTestId('dialog')).toHaveClass('hidden');
+    expect(screen.getByTestId('dialog-content')).toHaveClass('hidden');
   });
 
   it('should not close when dialog content clicked', async () => {
     // ARRANGE
     const user = userEvent.setup();
-    const closeFn = vi.fn();
     render(
-      <Dialog onClose={closeFn} isOpen={true}>
-        <div data-testid="content" />
+      <Dialog>
+        {({ close }) => (
+          <>
+            <Dialog.Trigger>Open</Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Title</Dialog.Title>
+                <Dialog.Subtitle>Subtitle</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>Body</Dialog.Body>
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </>
+        )}
       </Dialog>,
     );
-    await screen.findByTestId('dialog');
-    expect(screen.getByTestId('dialog')).not.toHaveClass('hidden');
+    await screen.findByTestId('dialog-trigger');
 
     // ACT
-    await user.click(screen.getByTestId('content'));
+    await user.click(screen.getByTestId('dialog-trigger'));
+    /* dialog is open */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).not.toHaveClass('hidden'));
+
+    await user.click(screen.getByTestId('dialog-body'));
 
     // ASSERT
-    expect(closeFn).not.toHaveBeenCalled();
-    expect(screen.getByTestId('dialog')).not.toHaveClass('hidden');
+    expect(screen.getByTestId('dialog-content')).not.toHaveClass('hidden');
   });
 });
diff --git a/src/common/components/Dialog/__tests__/DialogButton.test.tsx b/src/common/components/Dialog/__tests__/DialogButton.test.tsx
deleted file mode 100644
index e1467db..0000000
--- a/src/common/components/Dialog/__tests__/DialogButton.test.tsx
+++ /dev/null
@@ -1,53 +0,0 @@
-import { describe, expect, it } from 'vitest';
-
-import { render, screen } from 'test/test-utils';
-
-import DialogButton from '../DialogButton';
-
-describe('DialogButton', () => {
-  it('should render successfully', async () => {
-    // ARRANGE
-    render(<DialogButton>Submit</DialogButton>);
-    await screen.findByTestId('dialog-button');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-button')).toBeDefined();
-    expect(screen.getByTestId('dialog-button')).toHaveTextContent('Submit');
-  });
-
-  it('should use testId', async () => {
-    // ARRANGE
-    render(<DialogButton testId="my-dialog-button">Submit</DialogButton>);
-    await screen.findByTestId('my-dialog-button');
-
-    // ASSERT
-    expect(screen.getByTestId('my-dialog-button')).toBeDefined();
-  });
-
-  it('should use className', async () => {
-    // ARRANGE
-    render(<DialogButton className="my-class">Submit</DialogButton>);
-    await screen.findByTestId('dialog-button');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-button')).toHaveClass('my-class');
-  });
-
-  it('should render primary variant', async () => {
-    // ARRANGE
-    render(<DialogButton variant="primary">Submit</DialogButton>);
-    await screen.findByTestId('dialog-button');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-button')).toHaveClass('text-blue-600');
-  });
-
-  it('should render danger variant', async () => {
-    // ARRANGE
-    render(<DialogButton variant="danger">Submit</DialogButton>);
-    await screen.findByTestId('dialog-button');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-button')).toHaveClass('text-red-600');
-  });
-});
diff --git a/src/common/components/Dialog/__tests__/DialogContent.test.tsx b/src/common/components/Dialog/__tests__/DialogContent.test.tsx
deleted file mode 100644
index 64a412c..0000000
--- a/src/common/components/Dialog/__tests__/DialogContent.test.tsx
+++ /dev/null
@@ -1,34 +0,0 @@
-import { describe, expect, it } from 'vitest';
-
-import { render, screen } from 'test/test-utils';
-
-import DialogContent from '../DialogContent';
-
-describe('DialogContent', () => {
-  it('should render successfully', async () => {
-    // ARRANGE
-    render(<DialogContent />);
-    await screen.findByTestId('dialog-content');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-content')).toBeDefined();
-  });
-
-  it('should use testId', async () => {
-    // ARRANGE
-    render(<DialogContent testId="my-dialog-content" />);
-    await screen.findByTestId('my-dialog-content');
-
-    // ASSERT
-    expect(screen.getByTestId('my-dialog-content')).toBeDefined();
-  });
-
-  it('should use className', async () => {
-    // ARRANGE
-    render(<DialogContent className="my-class" />);
-    await screen.findByTestId('dialog-content');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-content')).toHaveClass('my-class');
-  });
-});
diff --git a/src/common/components/Dialog/__tests__/DialogHeading.test.tsx b/src/common/components/Dialog/__tests__/DialogHeading.test.tsx
deleted file mode 100644
index a4b34c8..0000000
--- a/src/common/components/Dialog/__tests__/DialogHeading.test.tsx
+++ /dev/null
@@ -1,34 +0,0 @@
-import { describe, expect, it } from 'vitest';
-
-import { render, screen } from 'test/test-utils';
-
-import DialogHeading from '../DialogHeading';
-
-describe('DialogHeading', () => {
-  it('should render successfully', async () => {
-    // ARRANGE
-    render(<DialogHeading />);
-    await screen.findByTestId('dialog-heading');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-heading')).toBeDefined();
-  });
-
-  it('should use testId', async () => {
-    // ARRANGE
-    render(<DialogHeading testId="my-dialog-heading" />);
-    await screen.findByTestId('my-dialog-heading');
-
-    // ASSERT
-    expect(screen.getByTestId('my-dialog-heading')).toBeDefined();
-  });
-
-  it('should use className', async () => {
-    // ARRANGE
-    render(<DialogHeading className="my-class" />);
-    await screen.findByTestId('dialog-heading');
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-heading')).toHaveClass('my-class');
-  });
-});
diff --git a/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx b/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx
index 5a7413b..70bcc7e 100644
--- a/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx
+++ b/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx
@@ -1,58 +1,100 @@
-import { ComponentPropsWithoutRef } from 'react';
-
 import { Task } from 'pages/Tasks/api/useGetUserTasks';
+import { useNavigate } from 'react-router-dom';
+import { useTranslation } from 'react-i18next';
+
+import { BaseComponentProps } from 'common/utils/types';
+import { useDeleteTask } from 'pages/Tasks/api/useDeleteTask';
+import { useToasts } from 'common/hooks/useToasts';
 import Dialog from 'common/components/Dialog/Dialog';
-import DialogHeading from 'common/components/Dialog/DialogHeading';
-import DialogContent from 'common/components/Dialog/DialogContent';
-import Divider from 'common/components/Divider/Divider';
-import DialogButtons from 'common/components/Dialog/DialogButtons';
-import DialogButton from 'common/components/Dialog/DialogButton';
+import FAIcon from 'common/components/Icon/FAIcon';
+import ErrorAlert from 'common/components/Alert/ErrorAlert';
 
 /**
  * Properties for the `TaskDeleteDialog` component.
- * @param {function} onCancel - A function called when the cancel button is clicked.
- * @param {function} onDelete - A function called when the delete button is clicked.
- * @param {Task} task - The `Task` being deleted.
- * @see {@link Dialog}
  */
-interface TaskDeleteDialogProps extends ComponentPropsWithoutRef<typeof Dialog> {
-  onCancel: () => void;
-  onDelete: () => void;
+interface TaskDeleteDialogProps extends BaseComponentProps {
   task: Task;
 }
 
 /**
  * The `TaskDeleteDialog` renders a dialog prompting for deletion confirmation
  * of a `Task`.
- * @param {TaskDeleteDialog} props - Component properties.
- * @returns {JSX.Element} JSX
  */
 const TaskDeleteDialog = ({
-  onCancel,
-  onDelete,
+  className,
   task,
   testId = 'dialog-task-delete',
-  ...dialogProps
 }: TaskDeleteDialogProps): JSX.Element => {
+  const { t } = useTranslation();
+  const navigate = useNavigate();
+  const { createToast } = useToasts();
+  const { mutate: deleteTask, isPending, error } = useDeleteTask();
+
+  /**
+   * Performs task deletion.
+   */
+  const doDelete = () => {
+    deleteTask(
+      { task },
+      {
+        onSuccess: () => {
+          createToast({
+            text: `Task deleted.`,
+            isAutoDismiss: true,
+          });
+          navigate(-1);
+        },
+      },
+    );
+  };
+
   return (
-    <Dialog testId={testId} {...dialogProps}>
-      <DialogHeading>Are you sure?</DialogHeading>
-      <DialogContent>
-        Deleting task <span className="text-neutral-500">{task.title}</span> is permanent.
-      </DialogContent>
-      <Divider />
-      <DialogButtons>
-        <DialogButton onClick={() => onCancel()} testId={`${testId}-button-cancel`}>
-          Cancel
-        </DialogButton>
-        <DialogButton
-          onClick={() => onDelete()}
-          variant="danger"
-          testId={`${testId}-button-delete`}
-        >
-          Delete
-        </DialogButton>
-      </DialogButtons>
+    <Dialog className={className} testId={testId}>
+      {({ close }) => (
+        <>
+          <Dialog.Trigger testId={`${testId}-trigger`}>
+            <FAIcon icon="trash" />
+          </Dialog.Trigger>
+          <Dialog.Content>
+            <Dialog.Header>
+              <Dialog.Title>Are you sure?</Dialog.Title>
+              <Dialog.Subtitle>Deleting a task is permanent.</Dialog.Subtitle>
+            </Dialog.Header>
+            <Dialog.Body>
+              {error && (
+                <ErrorAlert
+                  description={`${t('errors.unable-to-process')} ${error.message}`}
+                  className="mb-4"
+                  testId={`${testId}-error`}
+                />
+              )}
+              <div>
+                Delete task <span className="text-neutral-500">{task.title}</span>.
+              </div>
+            </Dialog.Body>
+            <Dialog.Separator />
+            <Dialog.Footer>
+              <Dialog.ButtonBar>
+                <Dialog.Button
+                  onClick={() => close()}
+                  disabled={isPending}
+                  testId={`${testId}-button-cancel`}
+                >
+                  Cancel
+                </Dialog.Button>
+                <Dialog.Button
+                  onClick={doDelete}
+                  variant="danger"
+                  disabled={isPending}
+                  testId={`${testId}-button-delete`}
+                >
+                  Delete
+                </Dialog.Button>
+              </Dialog.ButtonBar>
+            </Dialog.Footer>
+          </Dialog.Content>
+        </>
+      )}
     </Dialog>
   );
 };
diff --git a/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx b/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx
index ec55792..495b1cf 100644
--- a/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx
+++ b/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx
@@ -1,8 +1,10 @@
 import { describe, expect, it, vi } from 'vitest';
 import userEvent from '@testing-library/user-event';
 
-import { render, screen } from 'test/test-utils';
+import { render, screen, waitFor } from 'test/test-utils';
 import { todosFixture } from '__fixtures__/todos';
+import { ToastsContextValue } from 'common/providers/ToastsContext';
+import * as UseToasts from 'common/hooks/useToasts';
 
 import TaskDeleteDialog from '../../../../Tasks/components/Delete/TaskDeleteDialog';
 
@@ -10,69 +12,67 @@ describe('TaskDeleteDialog', () => {
   it('should render successfully', async () => {
     // ARRANGE
     const task = todosFixture[0];
-    const mockOnCancel = vi.fn();
-    const mockOnDelete = vi.fn();
-    render(
-      <TaskDeleteDialog
-        task={task}
-        onCancel={mockOnCancel}
-        onDelete={mockOnDelete}
-        testId="dialog"
-      />,
-    );
+    render(<TaskDeleteDialog task={task} testId="dialog" />);
     await screen.findByTestId('dialog');
 
     // ASSERT
     expect(screen.getByTestId('dialog')).toBeDefined();
   });
 
-  it('should call onCancel when cancel button clicked', async () => {
+  it('should close dialog when cancel button clicked', async () => {
     // ARRANGE
     const user = userEvent.setup();
     const task = todosFixture[0];
-    const mockOnCancel = vi.fn();
-    const mockOnDelete = vi.fn();
-    render(
-      <TaskDeleteDialog
-        task={task}
-        onCancel={mockOnCancel}
-        onDelete={mockOnDelete}
-        testId="dialog"
-      />,
-    );
+    render(<TaskDeleteDialog task={task} testId="dialog" />);
     await screen.findByTestId('dialog');
 
     // ACT
+    await user.click(screen.getByTestId('dialog-trigger'));
+    /* wait for dialog to open */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).not.toHaveClass('hidden'));
+
     await user.click(screen.getByTestId('dialog-button-cancel'));
+    /* wait for dialog to close */
+    await waitFor(() => expect(screen.getByTestId('dialog-content')).toHaveClass('hidden'));
 
     // ASSERT
-    expect(screen.getByTestId('dialog')).toBeDefined();
-    expect(mockOnCancel).toHaveBeenCalledOnce();
-    expect(mockOnDelete).not.toHaveBeenCalled();
+    expect(screen.getByTestId('dialog-content')).toHaveClass('hidden');
   });
 
-  it('should call onDelete when delete button clicked', async () => {
+  it('should delete task when delete button clicked', async () => {
     // ARRANGE
     const user = userEvent.setup();
     const task = todosFixture[0];
-    const mockOnCancel = vi.fn();
-    const mockOnDelete = vi.fn();
-    render(
-      <TaskDeleteDialog
-        task={task}
-        onCancel={mockOnCancel}
-        onDelete={mockOnDelete}
-        testId="dialog"
-      />,
-    );
+    const mockCreateToast = vi.fn();
+    const useToastsSpy = vi.spyOn(UseToasts, 'useToasts');
+    useToastsSpy.mockReturnValue({
+      createToast: mockCreateToast,
+      toasts: [],
+    } as unknown as ToastsContextValue);
+
+    render(<TaskDeleteDialog task={task} testId="dialog" />);
     await screen.findByTestId('dialog');
 
     // ACT
     await user.click(screen.getByTestId('dialog-button-delete'));
 
     // ASSERT
-    expect(screen.getByTestId('dialog')).toBeDefined();
-    expect(mockOnDelete).toHaveBeenCalledOnce();
-    expect(mockOnCancel).not.toHaveBeenCalled();
+    expect(mockCreateToast).toHaveBeenCalled();
+  });
+
+  it('should display error when there is a problem deleting task', async () => {
+    // ARRANGE
+    const user = userEvent.setup();
+    const task = { ...todosFixture[0], id: 999999 };
+
+    render(<TaskDeleteDialog task={task} testId="dialog" />);
+    await screen.findByTestId('dialog');
+
+    // ACT
+    await user.click(screen.getByTestId('dialog-button-delete'));
+    await waitFor(() => expect(screen.getByTestId('dialog-error')).toBeDefined());
+
+    // ASSERT
+    expect(screen.getByTestId('dialog-error')).toBeDefined();
   });
 });
diff --git a/src/pages/Tasks/components/TaskDetailLayout.tsx b/src/pages/Tasks/components/TaskDetailLayout.tsx
index 434eace..c0602d0 100644
--- a/src/pages/Tasks/components/TaskDetailLayout.tsx
+++ b/src/pages/Tasks/components/TaskDetailLayout.tsx
@@ -1,12 +1,8 @@
-import { useState } from 'react';
 import { useNavigate, useParams } from 'react-router-dom';
 import { useTranslation } from 'react-i18next';
 
 import { BaseComponentProps } from 'common/utils/types';
-import { Task } from 'pages/Tasks/api/useGetUserTasks';
 import { useGetTask } from '../api/useGetTask';
-import { useDeleteTask } from '../api/useDeleteTask';
-import { useToasts } from 'common/hooks/useToasts';
 import LoaderSkeleton from 'common/components/Loader/LoaderSkeleton';
 import FAIcon from 'common/components/Icon/FAIcon';
 import TaskDeleteDialog from './Delete/TaskDeleteDialog';
@@ -31,10 +27,8 @@ const TaskDetailLayout = ({
   testId = 'layout-task-detail',
 }: TaskDetailLayoutProps): JSX.Element => {
   const { t } = useTranslation();
-  const [isDeleteDialogOpen, setIsDeleteDialogOpen] = useState<boolean>(false);
   const navigate = useNavigate();
   const { taskId } = useParams();
-  const { createToast } = useToasts();
 
   const {
     data: task,
@@ -42,24 +36,6 @@ const TaskDetailLayout = ({
     isLoading: isLoadingTask,
   } = useGetTask({ taskId: Number(taskId) });
 
-  const { mutate: deleteTask, isPending: isDeletePending, error: deleteError } = useDeleteTask();
-
-  const doDelete = (task: Task) => {
-    setIsDeleteDialogOpen(false);
-    deleteTask(
-      { task },
-      {
-        onSuccess: () => {
-          createToast({
-            text: `Deleted task ${task.id}`,
-            isAutoDismiss: true,
-          });
-          navigate(-1);
-        },
-      },
-    );
-  };
-
   return (
     <div className={className} data-testid={testId}>
       {/* Heading */}
@@ -76,24 +52,18 @@ const TaskDetailLayout = ({
 
         {/* Menu */}
         <div className="ms-auto flex items-center gap-4">
-          <Button
-            variant="text"
-            size="icon"
-            title="Edit"
-            onClick={() => navigate('edit')}
-            testId={`${testId}-button-edit`}
-          >
-            <FAIcon icon="pencil" />
-          </Button>
-          <Button
-            variant="text"
-            size="icon"
-            title="Delete"
-            onClick={() => setIsDeleteDialogOpen(true)}
-            testId={`${testId}-button-delete`}
-          >
-            <FAIcon icon="trash" />
-          </Button>
+          {task && (
+            <Button
+              variant="text"
+              size="icon"
+              title="Edit"
+              onClick={() => navigate('edit')}
+              testId={`${testId}-button-edit`}
+            >
+              <FAIcon icon="pencil" />
+            </Button>
+          )}
+          {task && <TaskDeleteDialog task={task} />}
           <Button
             variant="text"
             size="icon"
@@ -115,16 +85,7 @@ const TaskDetailLayout = ({
         />
       )}
 
-      {deleteError && (
-        <ErrorAlert
-          title="Unable to delete task"
-          description={deleteError.message}
-          className="my-4"
-          testId={`${testId}-error-delete`}
-        />
-      )}
-
-      {(isLoadingTask || isDeletePending) && (
+      {isLoadingTask && (
         <div data-testid={`${testId}-loading`}>
           <div className="mt-4">
             <LoaderSkeleton className="mb-2 h-4 w-12" />
@@ -141,17 +102,9 @@ const TaskDetailLayout = ({
         </div>
       )}
 
-      {task && !isDeletePending && (
+      {task && (
         <div data-testid={`${testId}-task`}>
           <TaskView task={task} testId={`${testId}-task-view`} />
-
-          <TaskDeleteDialog
-            isOpen={isDeleteDialogOpen}
-            onCancel={() => setIsDeleteDialogOpen(false)}
-            onClose={() => setIsDeleteDialogOpen(false)}
-            onDelete={() => doDelete(task)}
-            task={task}
-          />
         </div>
       )}
     </div>
diff --git a/src/pages/Tasks/components/__tests__/TaskDetailLayout.test.tsx b/src/pages/Tasks/components/__tests__/TaskDetailLayout.test.tsx
index f4b6975..b42359e 100644
--- a/src/pages/Tasks/components/__tests__/TaskDetailLayout.test.tsx
+++ b/src/pages/Tasks/components/__tests__/TaskDetailLayout.test.tsx
@@ -1,9 +1,8 @@
 import { afterEach, describe, expect, it, vi } from 'vitest';
 import userEvent from '@testing-library/user-event';
-import { UseMutationResult, UseQueryResult } from '@tanstack/react-query';
+import { UseQueryResult } from '@tanstack/react-query';
 
 import { render, screen } from 'test/test-utils';
-import * as UseDeleteTask from 'pages/Tasks/api/useDeleteTask';
 import * as UseGetTask from 'pages/Tasks/api/useGetTask';
 import { Task } from 'pages/Tasks/api/useGetUserTasks';
 
@@ -102,82 +101,4 @@ describe('TaskDetailLayout', () => {
     expect(mockNavigate).toHaveBeenCalledTimes(1);
     expect(mockNavigate).toHaveBeenCalledWith('edit');
   });
-
-  it('should display task delete error', async () => {
-    // ARRANGE
-    const useDeleteTaskSpy = vi.spyOn(UseDeleteTask, 'useDeleteTask');
-    useDeleteTaskSpy.mockReturnValue({
-      data: undefined,
-      error: new Error(),
-      isLoading: false,
-    } as unknown as UseMutationResult<void, Error, UseDeleteTask.DeleteTaskVariables>);
-    render(<TaskDetailLayout testId="component" />);
-    await screen.findByTestId('component-error-delete');
-
-    // ASSERT
-    expect(screen.getByTestId('component-error-delete')).toBeDefined();
-  });
-
-  it('should show the delete dialog', async () => {
-    // ARRANGE
-    const user = userEvent.setup();
-    render(<TaskDetailLayout testId="component" />);
-    await screen.findByTestId('dialog-task-delete');
-    expect(screen.getByTestId('dialog-task-delete')).toHaveClass('hidden');
-
-    // ACT
-    await user.click(screen.getByTestId('component-button-delete'));
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-task-delete')).not.toHaveClass('hidden');
-  });
-
-  it('should hide the delete dialog when cancel clicked', async () => {
-    // ARRANGE
-    const user = userEvent.setup();
-    render(<TaskDetailLayout testId="component" />);
-    await screen.findByTestId('dialog-task-delete');
-    expect(screen.getByTestId('dialog-task-delete')).toHaveClass('hidden');
-
-    // ACT
-    await user.click(screen.getByTestId('component-button-delete'));
-    expect(screen.getByTestId('dialog-task-delete')).not.toHaveClass('hidden');
-
-    await user.click(screen.getByTestId('dialog-task-delete-button-cancel'));
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-task-delete')).toHaveClass('hidden');
-  });
-
-  it('should hide the delete dialog when backdrop clicked', async () => {
-    // ARRANGE
-    const user = userEvent.setup();
-    render(<TaskDetailLayout testId="component" />);
-    await screen.findByTestId('dialog-task-delete');
-    expect(screen.getByTestId('dialog-task-delete')).toHaveClass('hidden');
-
-    // ACT
-    await user.click(screen.getByTestId('component-button-delete'));
-    expect(screen.getByTestId('dialog-task-delete')).not.toHaveClass('hidden');
-
-    await user.click(screen.getByTestId('dialog-task-delete-backdrop'));
-
-    // ASSERT
-    expect(screen.getByTestId('dialog-task-delete')).toHaveClass('hidden');
-  });
-
-  it('should delete a task', async () => {
-    // ARRANGE
-    const user = userEvent.setup();
-    render(<TaskDetailLayout testId="component" />);
-    await screen.findByTestId('component-button-delete');
-
-    // ACT
-    await user.click(screen.getByTestId('component-button-delete'));
-    await user.click(screen.getByTestId('dialog-task-delete-button-delete'));
-
-    // ASSERT
-    expect(mockNavigate).toHaveBeenCalledTimes(1);
-    expect(mockNavigate).toHaveBeenCalledWith(-1);
-  });
 });

From 9649db06a68891d36185dbf9b7b5e1e7e3a818fc Mon Sep 17 00:00:00 2001
From: Matthew Warman <warmanmr@gmail.com>
Date: Tue, 4 Mar 2025 14:02:43 -0500
Subject: [PATCH 2/5] #57 stories

---
 src/common/components/Dialog/Dialog.tsx       |  2 +-
 .../Dialog/__stories__/Dialog.stories.tsx     | 99 +++++++++++++------
 .../__stories__/DialogButton.stories.tsx      | 48 ---------
 .../__stories__/DialogHeading.stories.tsx     | 36 -------
 4 files changed, 69 insertions(+), 116 deletions(-)
 delete mode 100644 src/common/components/Dialog/__stories__/DialogButton.stories.tsx
 delete mode 100644 src/common/components/Dialog/__stories__/DialogHeading.stories.tsx

diff --git a/src/common/components/Dialog/Dialog.tsx b/src/common/components/Dialog/Dialog.tsx
index 60afd84..f40b735 100644
--- a/src/common/components/Dialog/Dialog.tsx
+++ b/src/common/components/Dialog/Dialog.tsx
@@ -38,7 +38,7 @@ type DialogRenderFnContext = {
 type DialogRenderFn = (ctx: DialogRenderFnContext) => ReactNode;
 
 export interface DialogProps extends BaseComponentProps {
-  children: ReactNode | DialogRenderFn;
+  children?: ReactNode | DialogRenderFn;
 }
 
 /**
diff --git a/src/common/components/Dialog/__stories__/Dialog.stories.tsx b/src/common/components/Dialog/__stories__/Dialog.stories.tsx
index 6ec9400..4a2a80a 100644
--- a/src/common/components/Dialog/__stories__/Dialog.stories.tsx
+++ b/src/common/components/Dialog/__stories__/Dialog.stories.tsx
@@ -1,61 +1,98 @@
 import type { Meta, StoryObj } from '@storybook/react';
 
 import Dialog from '../Dialog';
-import DialogHeading from '../DialogHeading';
-import DialogContent from '../DialogContent';
-import Divider from '../../Divider/Divider';
-import DialogButton from '../DialogButton';
-import DialogButtons from '../DialogButtons';
+import Button from 'common/components/Button/Button';
 
 const meta = {
-  title: 'Common/Dialog/Dialog',
+  title: 'Common/Dialog',
   component: Dialog,
   parameters: {
-    layout: 'padded',
+    layout: 'centered',
   },
   tags: ['autodocs'],
   argTypes: {
-    children: { description: 'The content.' },
+    children: {
+      description: 'The dialog content. May contain a ReactNode or a render props function.',
+    },
     className: { description: 'Additional CSS classes.' },
-    isOpen: { description: 'Indicates if the dialog is shown. ', control: { type: 'boolean' } },
-    onClose: { description: 'Function called when the Dialog closes.' },
     testId: { description: 'The test identifier.' },
   },
-  args: {
-    isOpen: true,
-  },
 } satisfies Meta<typeof Dialog>;
 
 export default meta;
 
 type Story = StoryObj<typeof meta>;
 
-export const Delete: Story = {
+export const ConfirmDelete: Story = {
   render: (args) => (
     <Dialog {...args}>
-      <DialogHeading>Are you sure?</DialogHeading>
-      <DialogContent>Deletion is permanent.</DialogContent>
-      <Divider />
-      <DialogButtons>
-        <DialogButton>Cancel</DialogButton>
-        <DialogButton variant="danger">Delete</DialogButton>
-      </DialogButtons>
+      <Dialog.Trigger>
+        <Button>Open</Button>
+      </Dialog.Trigger>
+      <Dialog.Content>
+        <Dialog.Header>
+          <Dialog.Title>Are you sure?</Dialog.Title>
+          <Dialog.Subtitle>Deletion cannot be undone.</Dialog.Subtitle>
+        </Dialog.Header>
+        <Dialog.Body>
+          Delete task <span className="font-bold">18 Use React</span>.
+        </Dialog.Body>
+        <Dialog.Separator />
+        <Dialog.Footer>
+          <Dialog.ButtonBar>
+            <Dialog.Button>Cancel</Dialog.Button>
+            <Dialog.Button variant="danger">Delete</Dialog.Button>
+          </Dialog.ButtonBar>
+        </Dialog.Footer>
+      </Dialog.Content>
     </Dialog>
   ),
 };
 
-export const Info: Story = {
+export const LoremIpsum: Story = {
   render: (args) => (
     <Dialog {...args}>
-      <DialogHeading>Bob Smith</DialogHeading>
-      <DialogContent className="flex flex-col">
-        <div className="font-bold">Manager of Stories</div>
-        <div>Raleigh, NC</div>
-        <div>+1 123-456-7890</div>
-      </DialogContent>
-      <DialogButtons>
-        <DialogButton>Close</DialogButton>
-      </DialogButtons>
+      {({ close }) => (
+        <>
+          <Dialog.Trigger>
+            <Button>Open</Button>
+          </Dialog.Trigger>
+          <Dialog.Content>
+            <Dialog.Header>
+              <Dialog.Title>Amet at duis deserunt ad ad ornare.</Dialog.Title>
+              <Dialog.Subtitle>
+                Nam cupidatat duis dolore magna aute posuere. Esse arcu morbi quis consequat
+                facilisis lorem pulvinar.
+              </Dialog.Subtitle>
+            </Dialog.Header>
+            <Dialog.Body>
+              <div className="mb-2">
+                Cupidatat excepteur mi magna nisi sint. Sint officia donec duis egestas cupidatat
+                quam consectetur fermentum. Egestas incididunt esse magna ex occaecat nunc arcu.
+                Adipiscing sit laborum adipiscing aliqua tempor amet. Tempus laboris lorem deserunt
+                aute sint ullamco magna pariatur. Ipsum elit justo cupidatat ea a exercitation
+                pariatur.
+              </div>
+              <div>
+                Mi tempor incididunt duis ullamco morbi minim. Proin condimentum adipiscing officia
+                et ipsum nunc. Est et est quam labore sit velit. Eiusmod laboris facilisis aliquip
+                nulla aliqua occaecat quam mi. Ligula aute fermentum tempus dolore qui dolore culpa.
+                Quis quam ligula nulla exercitation exercitation nibh. Bibendum laboris mollit vitae
+                sunt elit aenean a non. Neque dui proident cupidatat proident morbi sunt sapien.
+              </div>
+            </Dialog.Body>
+            <Dialog.Separator />
+            <Dialog.Footer>
+              <Dialog.ButtonBar>
+                <Dialog.Button onClick={() => close()}>Cancel</Dialog.Button>
+                <Dialog.Button onClick={() => close()} variant="primary">
+                  Accept
+                </Dialog.Button>
+              </Dialog.ButtonBar>
+            </Dialog.Footer>
+          </Dialog.Content>
+        </>
+      )}
     </Dialog>
   ),
 };
diff --git a/src/common/components/Dialog/__stories__/DialogButton.stories.tsx b/src/common/components/Dialog/__stories__/DialogButton.stories.tsx
deleted file mode 100644
index 8253281..0000000
--- a/src/common/components/Dialog/__stories__/DialogButton.stories.tsx
+++ /dev/null
@@ -1,48 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react';
-
-import DialogButton from '../DialogButton';
-
-const meta = {
-  title: 'Common/Dialog/DialogButton',
-  component: DialogButton,
-  parameters: {
-    layout: 'centered',
-  },
-  tags: ['autodocs'],
-  argTypes: {
-    children: { description: 'The content.' },
-    className: { description: 'Additional CSS classes.' },
-    testId: { description: 'The test identifier.' },
-    variant: { description: 'The variant.' },
-  },
-  args: {
-    children: 'Label',
-    variant: 'primary',
-  },
-} satisfies Meta<typeof DialogButton>;
-
-export default meta;
-
-type Story = StoryObj<typeof meta>;
-
-export const Default: Story = {};
-
-export const Primary: Story = {
-  args: {
-    children: 'Primary',
-  },
-};
-
-export const Secondary: Story = {
-  args: {
-    children: 'Secondary',
-    variant: 'secondary',
-  },
-};
-
-export const Danger: Story = {
-  args: {
-    children: 'Danger',
-    variant: 'danger',
-  },
-};
diff --git a/src/common/components/Dialog/__stories__/DialogHeading.stories.tsx b/src/common/components/Dialog/__stories__/DialogHeading.stories.tsx
deleted file mode 100644
index 84cbd29..0000000
--- a/src/common/components/Dialog/__stories__/DialogHeading.stories.tsx
+++ /dev/null
@@ -1,36 +0,0 @@
-import type { Meta, StoryObj } from '@storybook/react';
-
-import DialogHeading from '../DialogHeading';
-
-const meta = {
-  title: 'Common/Dialog/DialogHeading',
-  component: DialogHeading,
-  parameters: {
-    layout: 'centered',
-  },
-  tags: ['autodocs'],
-  argTypes: {
-    children: { description: 'The content.' },
-    className: { description: 'Additional CSS classes.' },
-    testId: { description: 'The test identifier.' },
-  },
-  args: {
-    children: 'Are you sure?',
-  },
-} satisfies Meta<typeof DialogHeading>;
-
-export default meta;
-
-type Story = StoryObj<typeof meta>;
-
-export const Confirm: Story = {};
-
-export const Info: Story = {
-  args: {
-    children: (
-      <>
-        <span className="font-bold">About:</span> Widgets
-      </>
-    ),
-  },
-};

From 880aa67f3318fdf9364fa0524f61d53ee726a19b Mon Sep 17 00:00:00 2001
From: Matthew Warman <warmanmr@gmail.com>
Date: Tue, 4 Mar 2025 14:39:36 -0500
Subject: [PATCH 3/5] #57 DialogComponents page

---
 src/common/components/Router/Router.tsx       |   5 +
 src/pages/Components/ComponentsPage.tsx       |   3 +
 .../components/DialogComponents.tsx           | 229 ++++++++++++++++++
 .../__tests__/DialogComponents.test.tsx       |  32 +++
 4 files changed, 269 insertions(+)
 create mode 100644 src/pages/Components/components/DialogComponents.tsx
 create mode 100644 src/pages/Components/components/__tests__/DialogComponents.test.tsx

diff --git a/src/common/components/Router/Router.tsx b/src/common/components/Router/Router.tsx
index ab23907..c9abb73 100644
--- a/src/common/components/Router/Router.tsx
+++ b/src/common/components/Router/Router.tsx
@@ -20,6 +20,7 @@ import AvatarComponents from 'pages/Components/components/AvatarComponents';
 import BadgeComponents from 'pages/Components/components/BadgeComponents';
 import ButtonComponents from 'pages/Components/components/ButtonComponents';
 import CardComponents from 'pages/Components/components/CardComponents';
+import DialogComponents from 'pages/Components/components/DialogComponents';
 import DropdownComponents from 'pages/Components/components/DropdownComponents';
 import SearchInputComponents from 'pages/Components/components/SearchInputComponents';
 import TabsComponents from 'pages/Components/components/TabsComponents';
@@ -107,6 +108,10 @@ export const routes: RouteObject[] = [
                 path: 'card',
                 element: <CardComponents />,
               },
+              {
+                path: 'dialog',
+                element: <DialogComponents />,
+              },
               {
                 path: 'dropdown',
                 element: <DropdownComponents />,
diff --git a/src/pages/Components/ComponentsPage.tsx b/src/pages/Components/ComponentsPage.tsx
index be6d611..dfbe49b 100644
--- a/src/pages/Components/ComponentsPage.tsx
+++ b/src/pages/Components/ComponentsPage.tsx
@@ -35,6 +35,9 @@ const ComponentsPage = (): JSX.Element => {
             <MenuNavLink to="card" styleActive>
               Card
             </MenuNavLink>
+            <MenuNavLink to="dialog" styleActive>
+              Dialog
+            </MenuNavLink>
             <MenuNavLink to="dropdown" styleActive>
               Dropdown
             </MenuNavLink>
diff --git a/src/pages/Components/components/DialogComponents.tsx b/src/pages/Components/components/DialogComponents.tsx
new file mode 100644
index 0000000..9048852
--- /dev/null
+++ b/src/pages/Components/components/DialogComponents.tsx
@@ -0,0 +1,229 @@
+import { createColumnHelper } from '@tanstack/react-table';
+
+import { BaseComponentProps } from 'common/utils/types';
+import { ComponentProperty } from '../model/components';
+import Table from 'common/components/Table/Table';
+import CodeSnippet from 'common/components/Text/CodeSnippet';
+import Heading from 'common/components/Text/Heading';
+import Dialog from 'common/components/Dialog/Dialog';
+import Button from 'common/components/Button/Button';
+
+/**
+ * The `DialogComponents` React component renders a set of examples illustrating
+ * the use of the `Dialog` component.
+ */
+const DialogComponents = ({
+  className,
+  testId = 'components-dialog',
+}: BaseComponentProps): JSX.Element => {
+  const data: ComponentProperty[] = [
+    {
+      name: 'children',
+      description: 'The content to be displayed. May be a ReactNode or a render prop function.',
+    },
+    {
+      name: 'className',
+      description: 'Optional. Additional CSS class names.',
+    },
+    {
+      name: 'testId',
+      description: 'Optional. Identifier for testing.',
+    },
+  ];
+  const columnHelper = createColumnHelper<ComponentProperty>();
+  const columns = [
+    columnHelper.accessor('name', {
+      cell: (info) => (
+        <span className="font-mono text-sky-700 dark:text-sky-500">{info.getValue()}</span>
+      ),
+      header: () => 'Name',
+    }),
+    columnHelper.accessor('description', {
+      cell: (info) => info.renderValue(),
+      header: () => 'Description',
+    }),
+  ];
+
+  return (
+    <section className={className} data-testid={testId}>
+      <Heading level={2} className="mb-4">
+        Dialog Component
+      </Heading>
+
+      <div className="my-8">
+        The <span className="font-mono font-bold">Dialog</span> is a modal window that displays on
+        top of the main content, typically asking the user to take an action or confirm a decision.
+        Compose a Dialog using combinations of: <code>Trigger</code>, <code>Content</code>,{' '}
+        <code>Header</code>, <code>Title</code>, <code>Subtitle</code>, <code>Body</code>,{' '}
+        <code>Footer</code>, <code>ButtonBar</code>, and <code>Button</code>.
+      </div>
+
+      <div className="my-8">
+        <Heading level={3} className="mb-2">
+          Properties
+        </Heading>
+        <Table<ComponentProperty, string> data={data} columns={columns} />
+      </div>
+
+      <Heading level={3}>Examples</Heading>
+
+      <Heading level={4}>Dialog with ReactNode children</Heading>
+      <div className="my-8">
+        <div className="mb-2 flex place-content-center rounded-sm border border-neutral-500/10 p-4 dark:bg-neutral-700/25">
+          <Dialog>
+            <Dialog.Trigger>
+              <Button>Open</Button>
+            </Dialog.Trigger>
+            <Dialog.Content>
+              <Dialog.Header>
+                <Dialog.Title>Are you sure?</Dialog.Title>
+                <Dialog.Subtitle>Deletion is permanent and cannot be undone.</Dialog.Subtitle>
+              </Dialog.Header>
+              <Dialog.Body>
+                Delete issue:{' '}
+                <span className="font-bold text-neutral-500">
+                  987 Use dialog to confirm task delete
+                </span>
+              </Dialog.Body>
+              <Dialog.Separator />
+              <Dialog.Footer>
+                <Dialog.ButtonBar>
+                  <Dialog.Button>Cancel</Dialog.Button>
+                  <Dialog.Button variant="danger">Delete</Dialog.Button>
+                </Dialog.ButtonBar>
+              </Dialog.Footer>
+            </Dialog.Content>
+          </Dialog>
+        </div>
+        <CodeSnippet
+          className="my-2"
+          code={`<Dialog>
+  <Dialog.Trigger>
+    <Button>Open</Button>
+  </Dialog.Trigger>
+  <Dialog.Content>
+    <Dialog.Header>
+      <Dialog.Title>Are you sure?</Dialog.Title>
+      <Dialog.Subtitle>Deletion is permanent and cannot be undone.</Dialog.Subtitle>
+    </Dialog.Header>
+    <Dialog.Body>
+      Delete issue:{' '}
+      <span className="font-bold text-neutral-500">
+        987 Use dialog to confirm task delete
+      </span>
+    </Dialog.Body>
+    <Dialog.Separator />
+    <Dialog.Footer>
+      <Dialog.ButtonBar>
+        <Dialog.Button>Cancel</Dialog.Button>
+        <Dialog.Button variant="danger">Delete</Dialog.Button>
+      </Dialog.ButtonBar>
+    </Dialog.Footer>
+  </Dialog.Content>
+</Dialog>`}
+        />
+      </div>
+
+      <Heading level={4}>Dialog with render props</Heading>
+      <div className="my-8">
+        <div className="mb-2 flex place-content-center rounded-sm border border-neutral-500/10 p-4 dark:bg-neutral-700/25">
+          <Dialog>
+            {({ close }) => (
+              <>
+                <Dialog.Trigger>
+                  <Button>Open</Button>
+                </Dialog.Trigger>
+                <Dialog.Content>
+                  <Dialog.Header>
+                    <Dialog.Title>Terms and Conditions</Dialog.Title>
+                    <Dialog.Subtitle>
+                      Nam cupidatat duis dolore magna aute posuere. Esse arcu morbi quis consequat
+                      facilisis lorem pulvinar.
+                    </Dialog.Subtitle>
+                  </Dialog.Header>
+                  <Dialog.Body>
+                    <div className="mb-2">
+                      Dolor proident aliqua ornare consectetur. Sapien est do quam labore qos
+                      veniam. Aute sint sunt commodo ea e. Ligula anim amet nulla morbi nulla.
+                      Laborum duis sunt exercitation justo. Quis aliquip posuere incididunt et
+                      consectetur. Nostrud incididunt laborum pulvinar ea ut ex at.
+                    </div>
+                    <div>
+                      Pariatur anim veniam morbi dui consectetur officia. Occaecat e dolore mi enim
+                      morbi aliquip e. Condimentum adipiscing sunt commodo proident enim laborum mi.
+                      Dui labore posuere ex exercitation justo morbi. A consequat pulvinar aliqua do
+                      duis ligula. Ullamco enim condimentum pariatur dolor nulla ad quam. Nam
+                      egestas laboris laoreet et ipsum deserunt ligula.
+                    </div>
+                  </Dialog.Body>
+                  <Dialog.Separator />
+                  <Dialog.Footer>
+                    <Dialog.ButtonBar>
+                      <Dialog.Button onClick={() => close()} testId="dialog-button-decline">
+                        Decline
+                      </Dialog.Button>
+                      <Dialog.Button
+                        variant="primary"
+                        onClick={() => close()}
+                        testId="dialog-button-accept"
+                      >
+                        Accept
+                      </Dialog.Button>
+                    </Dialog.ButtonBar>
+                  </Dialog.Footer>
+                </Dialog.Content>
+              </>
+            )}
+          </Dialog>
+        </div>
+        <CodeSnippet
+          className="my-2"
+          code={`<Dialog>
+  {({ close }) => (
+    <>
+      <Dialog.Trigger>
+        <Button>Open</Button>
+      </Dialog.Trigger>
+      <Dialog.Content>
+        <Dialog.Header>
+          <Dialog.Title>Terms and Conditions</Dialog.Title>
+          <Dialog.Subtitle>
+            Nam cupidatat duis dolore magna aute posuere. Esse arcu morbi quis consequat
+            facilisis lorem pulvinar.
+          </Dialog.Subtitle>
+        </Dialog.Header>
+        <Dialog.Body>
+          <div className="mb-2">
+            Dolor proident aliqua ornare consectetur. Sapien est do quam labore qos
+            veniam. Aute sint sunt commodo ea e. Ligula anim amet nulla morbi nulla.
+            Laborum duis sunt exercitation justo. Quis aliquip posuere incididunt et
+            consectetur. Nostrud incididunt laborum pulvinar ea ut ex at.
+          </div>
+          <div>
+            Pariatur anim veniam morbi dui consectetur officia. Occaecat e dolore mi enim
+            morbi aliquip e. Condimentum adipiscing sunt commodo proident enim laborum mi.
+            Dui labore posuere ex exercitation justo morbi. A consequat pulvinar aliqua do
+            duis ligula. Ullamco enim condimentum pariatur dolor nulla ad quam. Nam
+            egestas laboris laoreet et ipsum deserunt ligula.
+          </div>
+        </Dialog.Body>
+        <Dialog.Separator />
+        <Dialog.Footer>
+          <Dialog.ButtonBar>
+            <Dialog.Button onClick={() => close()}>Decline</Dialog.Button>
+            <Dialog.Button variant="primary" onClick={() => close()}>
+              Accept
+            </Dialog.Button>
+          </Dialog.ButtonBar>
+        </Dialog.Footer>
+      </Dialog.Content>
+    </>
+  )}
+</Dialog>`}
+        />
+      </div>
+    </section>
+  );
+};
+
+export default DialogComponents;
diff --git a/src/pages/Components/components/__tests__/DialogComponents.test.tsx b/src/pages/Components/components/__tests__/DialogComponents.test.tsx
new file mode 100644
index 0000000..2130aa5
--- /dev/null
+++ b/src/pages/Components/components/__tests__/DialogComponents.test.tsx
@@ -0,0 +1,32 @@
+import { describe, expect, it } from 'vitest';
+
+import { render, screen } from 'test/test-utils';
+
+import DialogComponents from '../DialogComponents';
+import userEvent from '@testing-library/user-event';
+
+describe('DialogComponents', () => {
+  it('should render successfully', async () => {
+    // ARRANGE
+    render(<DialogComponents />);
+    await screen.findByTestId('components-dialog');
+
+    // ASSERT
+    expect(screen.getByTestId('components-dialog')).toBeDefined();
+  });
+
+  it('should click items', async () => {
+    // ARRANGE
+    const user = userEvent.setup();
+    render(<DialogComponents />);
+    await screen.findByTestId('components-dialog');
+
+    // ACT
+    /* exercising click handlers; normally would assert handler function called */
+    await user.click(screen.getByTestId('dialog-button-decline'));
+    await user.click(screen.getByTestId('dialog-button-accept'));
+
+    // ASSERT
+    expect(screen.getByTestId('components-dialog')).toBeDefined();
+  });
+});

From bb8fce45d9cc68e3f30b7fe4571a8779ecd9ca0e Mon Sep 17 00:00:00 2001
From: Matthew Warman <warmanmr@gmail.com>
Date: Wed, 5 Mar 2025 06:15:12 -0500
Subject: [PATCH 4/5] #57 pr fixes

---
 src/common/components/Dialog/Dialog.tsx | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/common/components/Dialog/Dialog.tsx b/src/common/components/Dialog/Dialog.tsx
index f40b735..f3e04b6 100644
--- a/src/common/components/Dialog/Dialog.tsx
+++ b/src/common/components/Dialog/Dialog.tsx
@@ -25,8 +25,8 @@ const DialogContext = createContext<DialogContextValue>({
 });
 
 /**
- * Defines the properties of the `Dialog` render prop function context.
- * @param close - Close the dialog.
+ * Defines the properties of the `Dialog` render prop function context object.
+ * @param close - Closes the dialog.
  */
 type DialogRenderFnContext = {
   close: () => void;
@@ -258,7 +258,7 @@ export interface ButtonProps extends Omit<CommonButtonProps, 'variant'>, ButtonV
 const Button = ({
   className,
   variant = 'secondary',
-  testId = 'dialog-footer',
+  testId = 'dialog-button',
   ...props
 }: ButtonProps): JSX.Element => {
   return (

From 20c2b18a425b87d9fe95e77309c50b0bd867717d Mon Sep 17 00:00:00 2001
From: Matthew Warman <warmanmr@gmail.com>
Date: Wed, 5 Mar 2025 06:26:00 -0500
Subject: [PATCH 5/5] #57 pr fixes

---
 .../components/Delete/TaskDeleteDialog.tsx    | 11 ++++-----
 .../__tests__/TaskDeleteDialog.test.tsx       | 24 +++++++++++++++----
 .../Tasks/components/TaskDetailLayout.tsx     |  6 ++++-
 3 files changed, 30 insertions(+), 11 deletions(-)

diff --git a/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx b/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx
index 70bcc7e..2ce606c 100644
--- a/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx
+++ b/src/pages/Tasks/components/Delete/TaskDeleteDialog.tsx
@@ -1,18 +1,18 @@
-import { Task } from 'pages/Tasks/api/useGetUserTasks';
+import { PropsWithChildren } from 'react';
 import { useNavigate } from 'react-router-dom';
 import { useTranslation } from 'react-i18next';
 
 import { BaseComponentProps } from 'common/utils/types';
+import { Task } from 'pages/Tasks/api/useGetUserTasks';
 import { useDeleteTask } from 'pages/Tasks/api/useDeleteTask';
 import { useToasts } from 'common/hooks/useToasts';
 import Dialog from 'common/components/Dialog/Dialog';
-import FAIcon from 'common/components/Icon/FAIcon';
 import ErrorAlert from 'common/components/Alert/ErrorAlert';
 
 /**
  * Properties for the `TaskDeleteDialog` component.
  */
-interface TaskDeleteDialogProps extends BaseComponentProps {
+interface TaskDeleteDialogProps extends BaseComponentProps, PropsWithChildren {
   task: Task;
 }
 
@@ -21,6 +21,7 @@ interface TaskDeleteDialogProps extends BaseComponentProps {
  * of a `Task`.
  */
 const TaskDeleteDialog = ({
+  children,
   className,
   task,
   testId = 'dialog-task-delete',
@@ -52,9 +53,7 @@ const TaskDeleteDialog = ({
     <Dialog className={className} testId={testId}>
       {({ close }) => (
         <>
-          <Dialog.Trigger testId={`${testId}-trigger`}>
-            <FAIcon icon="trash" />
-          </Dialog.Trigger>
+          <Dialog.Trigger testId={`${testId}-trigger`}>{children}</Dialog.Trigger>
           <Dialog.Content>
             <Dialog.Header>
               <Dialog.Title>Are you sure?</Dialog.Title>
diff --git a/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx b/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx
index 495b1cf..1c1516e 100644
--- a/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx
+++ b/src/pages/Tasks/components/Delete/__tests__/TaskDeleteDialog.test.tsx
@@ -12,7 +12,11 @@ describe('TaskDeleteDialog', () => {
   it('should render successfully', async () => {
     // ARRANGE
     const task = todosFixture[0];
-    render(<TaskDeleteDialog task={task} testId="dialog" />);
+    render(
+      <TaskDeleteDialog task={task} testId="dialog">
+        Open
+      </TaskDeleteDialog>,
+    );
     await screen.findByTestId('dialog');
 
     // ASSERT
@@ -23,7 +27,11 @@ describe('TaskDeleteDialog', () => {
     // ARRANGE
     const user = userEvent.setup();
     const task = todosFixture[0];
-    render(<TaskDeleteDialog task={task} testId="dialog" />);
+    render(
+      <TaskDeleteDialog task={task} testId="dialog">
+        Open
+      </TaskDeleteDialog>,
+    );
     await screen.findByTestId('dialog');
 
     // ACT
@@ -50,7 +58,11 @@ describe('TaskDeleteDialog', () => {
       toasts: [],
     } as unknown as ToastsContextValue);
 
-    render(<TaskDeleteDialog task={task} testId="dialog" />);
+    render(
+      <TaskDeleteDialog task={task} testId="dialog">
+        Open
+      </TaskDeleteDialog>,
+    );
     await screen.findByTestId('dialog');
 
     // ACT
@@ -65,7 +77,11 @@ describe('TaskDeleteDialog', () => {
     const user = userEvent.setup();
     const task = { ...todosFixture[0], id: 999999 };
 
-    render(<TaskDeleteDialog task={task} testId="dialog" />);
+    render(
+      <TaskDeleteDialog task={task} testId="dialog">
+        Open
+      </TaskDeleteDialog>,
+    );
     await screen.findByTestId('dialog');
 
     // ACT
diff --git a/src/pages/Tasks/components/TaskDetailLayout.tsx b/src/pages/Tasks/components/TaskDetailLayout.tsx
index c0602d0..e54be11 100644
--- a/src/pages/Tasks/components/TaskDetailLayout.tsx
+++ b/src/pages/Tasks/components/TaskDetailLayout.tsx
@@ -63,7 +63,11 @@ const TaskDetailLayout = ({
               <FAIcon icon="pencil" />
             </Button>
           )}
-          {task && <TaskDeleteDialog task={task} />}
+          {task && (
+            <TaskDeleteDialog task={task}>
+              <FAIcon icon="trash" />
+            </TaskDeleteDialog>
+          )}
           <Button
             variant="text"
             size="icon"