Skip to content

Commit bb6cc54

Browse files
committed
fix: delete dialog UI
1 parent 7e5d030 commit bb6cc54

File tree

4 files changed

+29
-86
lines changed

4 files changed

+29
-86
lines changed

apps/dashboard/app/(main)/websites/[id]/settings/_components/delete-dialog.tsx

Lines changed: 0 additions & 75 deletions
This file was deleted.

apps/dashboard/app/(main)/websites/[id]/settings/general/page.tsx

Lines changed: 25 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,17 +5,18 @@ import {
55
GearIcon,
66
PencilSimpleIcon,
77
TrashIcon,
8+
WarningCircleIcon,
89
} from "@phosphor-icons/react";
910
import { useParams, useRouter } from "next/navigation";
1011
import { useCallback, useState } from "react";
1112
import { toast } from "sonner";
1213
import { Button } from "@/components/ui/button";
14+
import { DeleteDialog } from "@/components/ui/delete-dialog";
1315
import { Label } from "@/components/ui/label";
1416
import { WebsiteDialog } from "@/components/website-dialog";
1517
import { useDeleteWebsite, useWebsite } from "@/hooks/use-websites";
1618
import { PageHeader } from "../../../_components/page-header";
1719
import { TOAST_MESSAGES } from "../../_components/shared/tracking-constants";
18-
import { DeleteWebsiteDialog } from "../_components/delete-dialog";
1920

2021
export default function GeneralSettingsPage() {
2122
const params = useParams();
@@ -158,13 +159,30 @@ export default function GeneralSettingsPage() {
158159
open={showEditDialog}
159160
website={websiteData}
160161
/>
161-
<DeleteWebsiteDialog
162+
<DeleteDialog
163+
confirmLabel="Delete Website"
164+
description={`Are you sure you want to delete ${websiteData.name || websiteData.domain}?`}
162165
isDeleting={deleteWebsiteMutation.isPending}
163-
onConfirmDelete={handleDeleteWebsite}
164-
onOpenChange={setShowDeleteDialog}
165-
open={showDeleteDialog}
166-
websiteData={websiteData}
167-
/>
166+
isOpen={showDeleteDialog}
167+
itemName={websiteData.name || websiteData.domain}
168+
onClose={() => setShowDeleteDialog(false)}
169+
onConfirm={handleDeleteWebsite}
170+
title="Delete Website"
171+
>
172+
<div className="rounded-md bg-secondary p-3 text-sm">
173+
<div className="flex items-start gap-2">
174+
<WarningCircleIcon className="h-5 w-5 shrink-0" />
175+
<div className="space-y-1">
176+
<p className="font-medium">Warning:</p>
177+
<ul className="list-disc space-y-1 pl-4 text-xs">
178+
<li>All analytics data will be permanently deleted</li>
179+
<li>Tracking will stop immediately</li>
180+
<li>All website settings will be lost</li>
181+
</ul>
182+
</div>
183+
</div>
184+
</div>
185+
</DeleteDialog>
168186
</div>
169187
);
170188
}

apps/dashboard/components/ui/delete-dialog.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export function DeleteDialog({
7676
weight="duotone"
7777
/>
7878
</div>
79-
<p className="text-muted-foreground text-sm">
79+
<p className="text-foreground text-sm">
8080
This action cannot be undone.
8181
</p>
8282
</div>
@@ -90,7 +90,7 @@ export function DeleteDialog({
9090
weight="duotone"
9191
/>
9292
</div>
93-
<p className="text-muted-foreground text-sm">
93+
<p className="text-foreground text-sm">
9494
This action cannot be undone.
9595
</p>
9696
</div>

apps/dashboard/components/ui/dialog.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ function DialogOverlay({
3737
return (
3838
<DialogPrimitive.Overlay
3939
className={cn(
40-
'data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 fixed inset-0 z-50 bg-black/50 data-[state=closed]:animate-out data-[state=open]:animate-in',
40+
'data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 fixed inset-0 z-[120] bg-black/50 data-[state=closed]:animate-out data-[state=open]:animate-in',
4141
className
4242
)}
4343
data-slot="dialog-overlay"
@@ -59,7 +59,7 @@ function DialogContent({
5959
<DialogOverlay />
6060
<DialogPrimitive.Content
6161
className={cn(
62-
'data-[state=closed]:fade-out-0 overflow-hidden data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 fixed top-[50%] left-[50%] z-50 grid w-full max-w-[calc(100%-2rem)] translate-x-[-50%] translate-y-[-50%] gap-5 rounded-lg border bg-card p-5 shadow-lg duration-200 data-[state=closed]:animate-out data-[state=open]:animate-in sm:max-w-lg',
62+
'data-[state=closed]:fade-out-0 overflow-hidden data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 fixed top-[50%] left-[50%] z-[130] grid w-full max-w-[calc(100%-2rem)] translate-x-[-50%] translate-y-[-50%] gap-5 rounded-lg border bg-card p-5 shadow-lg duration-200 data-[state=closed]:animate-out data-[state=open]:animate-in sm:max-w-lg',
6363
className
6464
)}
6565
data-slot="dialog-content"

0 commit comments

Comments
 (0)