Skip to content

fix: Simplified dialog animations by removing slide transitions - #378

Merged
bdjulbic merged 1 commit into
mainfrom
fix/dialog-zoom-in-instead-slide
Feb 23, 2026
Merged

fix: Simplified dialog animations by removing slide transitions#378
bdjulbic merged 1 commit into
mainfrom
fix/dialog-zoom-in-instead-slide

Conversation

@bdjulbic

@bdjulbic bdjulbic commented Feb 23, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Center dialog instead of sliding it from left to center

Summary by CodeRabbit

  • Style
    • Simplified dialog animations by removing slide transitions while preserving fade and zoom effects for a cleaner animation experience.

@coderabbitai

coderabbitai Bot commented Feb 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This PR simplifies the DialogContent animation className in the dialog component by removing specific slide-out/slide-in Tailwind transitions (slide-out-to-left-1/2, slide-out-to-top-[48%], slide-in-from-left-1/2, slide-in-from-top-[48%]) while preserving fade, zoom, and base animate-in/animate-out effects.

Changes

Cohort / File(s) Summary
Animation Simplification
packages/ui/src/components/dialog.tsx		 Removed slide-out/slide-in animation transitions from DialogContent while retaining fade, zoom, and core animate-in/animate-out animations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Dialogs once slid left and right,
Now they fade with gentle light,
Simpler animations, cleaner flow—
Less is more, this much we know! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title focuses on removing slide transitions, but the PR objective emphasizes centering the dialog with zoom instead of slide—a different framing of the same change.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/dialog-zoom-in-instead-slide

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ilyabo ilyabo changed the title fix: Center dialog popup fix: Simplified dialog animations by removing slide transitions Feb 23, 2026
@bdjulbic
bdjulbic merged commit f586488 into main Feb 23, 2026
5 checks passed
@bdjulbic
bdjulbic deleted the fix/dialog-zoom-in-instead-slide branch February 23, 2026 13:43
This was referenced May 7, 2026
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ilyabo ilyabo ilyabo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

See More

Successfully merging this pull request may close these issues.

2 participants

@bdjulbic @ilyabo