Skip to content

Refactor Dialog Slots#56

Merged
Shrinks99 merged 2 commits into
mainfrom
dialog-refactor
Jan 26, 2026
Merged

Refactor Dialog Slots#56
Shrinks99 merged 2 commits into
mainfrom
dialog-refactor

Conversation

@Shrinks99
Copy link
Copy Markdown
Member

Now follows this pattern:

<Dialog>
  <DialogTrigger>
    <Button>Open Dialog</Button>
  </DialogTrigger>

  <DialogContainer size="md">
    <DialogHeader>
      <DialogTitle>Dialog Title</DialogTitle>
      <DialogDescription>Optional subtitle or description</DialogDescription>
    </DialogHeader>

    <DialogContent>
      {/* Main content area - scrollable if needed */}
      <p>Your main dialog content goes here...</p>
    </DialogContent>

    <DialogFooter>
      <DialogClose asChild>
        <Button variant="outline">Cancel</Button>
      </DialogClose>
      <Button>Confirm</Button>
    </DialogFooter>
  </DialogContainer>
</Dialog>

Shrinks99 and others added 2 commits January 26, 2026 13:09
Now follows this pattern:

```tsx
<Dialog>
  <DialogTrigger>
    <Button>Open Dialog</Button>
  </DialogTrigger>

  <DialogContainer size="md">
    <DialogHeader>
      <DialogTitle>Dialog Title</DialogTitle>
      <DialogDescription>Optional subtitle or description</DialogDescription>
    </DialogHeader>

    <DialogContent>
      {/* Main content area - scrollable if needed */}
      <p>Your main dialog content goes here...</p>
    </DialogContent>

    <DialogFooter>
      <DialogClose asChild>
        <Button variant="outline">Cancel</Button>
      </DialogClose>
      <Button>Confirm</Button>
    </DialogFooter>
  </DialogContainer>
</Dialog>
```
Co-Authored-By: Claude <81847+claude@users.noreply.github.com>
@Shrinks99 Shrinks99 merged commit 26d7116 into main Jan 26, 2026
1 check passed
@Shrinks99 Shrinks99 deleted the dialog-refactor branch January 26, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant