yiekheng/cm_whatsapp_bot_v1
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(web): drop redundant Close button from account dialogs

Browse Source 
DialogFooter showCloseButton was rendering a third button (Close) next
to the Cancel + 'Yes, delete' / 'Yes, unpair' pair. The corner X icon
already closes the dialog, so the extra button was just visual noise.
Drop the prop on both account-card dialogs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
yiekheng 2026-05-10 18:57:06 +08:00
parent 6759ca8131
commit dbdb156a09
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
apps/web/src/app/accounts/[id]/delete-account-card.tsx
View File

@ -77,7 +77,7 @@ export function DeleteAccountCard({
cannot be undone. cannot be undone.
</DialogDescription> </DialogDescription>
</DialogHeader> </DialogHeader>
<DialogFooter showCloseButton> <DialogFooter>
<DialogClose asChild> <DialogClose asChild>
<Button type="button" variant="ghost" size="sm"> <Button type="button" variant="ghost" size="sm">
Cancel Cancel

2
apps/web/src/app/accounts/[id]/unpair-account-card.tsx
View File

@ -76,7 +76,7 @@ export function UnpairAccountCard({
deleted. deleted.
</DialogDescription> </DialogDescription>
</DialogHeader> </DialogHeader>
<DialogFooter showCloseButton> <DialogFooter>
<DialogClose asChild> <DialogClose asChild>
<Button type="button" variant="ghost" size="sm"> <Button type="button" variant="ghost" size="sm">
Cancel Cancel