Skip to content

[01360] Sheet animation should slide from direction instead of dialog-style scale#2900

Merged
nielsbosma merged 1 commit intomainfrom
plan-01360-Ivy-Framework
Apr 1, 2026
Merged

[01360] Sheet animation should slide from direction instead of dialog-style scale#2900
nielsbosma merged 1 commit intomainfrom
plan-01360-Ivy-Framework

Conversation

@nielsbosma
Copy link
Copy Markdown
Contributor

Summary

Fixed sheet directional slide animations that were completely broken due to CSS selectors targeting [data-radix-dialog-content], an attribute not set by Radix UI v1.1.15. Replaced all selectors with [data-sheet-side] which is already set on the SheetContent element. Also removed the alert-animate-enter class from SheetWidget and deleted unused .sheet-animate-enter/.sheet-animate-exit utility classes and overlay animation selectors.

API Changes

None.

Files Modified

  • src/frontend/src/widgets/sheet/sheet.css — Replaced [data-radix-dialog-content] selectors with [data-sheet-side] selectors, removed overlay animation selectors and unused utility classes
  • src/frontend/src/widgets/sheet/SheetWidget.tsx — Removed alert-animate-enter from className

Commits

  • e33c143 [01360] Fix sheet directional slide animation selectors

Artifacts

Screenshots

basic-before-open

directions-initial

events-after-close

events-sheet-open

integration-item-selected

integration-sheet-open

withsheet-initial

Replace broken [data-radix-dialog-content] CSS selectors with
[data-sheet-side] which is actually set on the SheetContent element.
Remove alert-animate-enter class, unused overlay animation CSS,
and unused .sheet-animate-enter/.sheet-animate-exit classes.
@nielsbosma nielsbosma merged commit 994e102 into main Apr 1, 2026
11 checks passed
@nielsbosma nielsbosma deleted the plan-01360-Ivy-Framework branch April 1, 2026 09:20
@artem-ivy-ai
Copy link
Copy Markdown
Collaborator

Deploy timed out

I'm preparing your docs & samples for this PR. I'll update the comment as Sliplane reports progress.

Logs

No terminal state received from Sliplane after 30 minutes.

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.

2 participants