Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sheet component's closing animation is missing #68

Closed
declanchiu opened this issue Mar 27, 2024 · 7 comments
Closed

Sheet component's closing animation is missing #68

declanchiu opened this issue Mar 27, 2024 · 7 comments

Comments

@declanchiu
Copy link
Contributor

I realized that the closing animation of the Sheet component is missing. The sheet component in shadcn/ui has a collapsing effect when it is closed.

@declanchiu
Copy link
Contributor Author

declanchiu commented Mar 27, 2024

I took a look at the source code for solid-ui's sheet component and I think I've figured out why.

image

Currently there is only the animate-in effect of twindcss-animate. The animate-out effect is missing。

The expand and close effects of the sheet component above shadcn/ui are controlled by the data-* property selector.

image

But soliid-ui's sheet component's primitive component is kobalte/Dialog, and its data-* attribute parameter is not explicitly given a value. So even if you add something like shadcn/ui's data-[epanded=true]: animatie-in, it won't work.

image

@declanchiu
Copy link
Contributor Author

I think I would be interested in solving this problem and contributing a pr... 😃

Do you have a better suggestion? @sek-consulting

@jcramb
Copy link
Contributor

jcramb commented Mar 28, 2024

@declanchiu please do :) I need to fix this for my current project and would appreciate it!

@stefan-karger
Copy link
Owner

@declanchiu If you want to contribute feel free to do so :)

I just merged the big changes back into the main branch so you can use that as a base without any worries.

@declanchiu
Copy link
Contributor Author

@declanchiu If you want to contribute feel free to do so :) 如果您想做出贡献,请随时这样做:)

I just merged the big changes back into the main branch so you can use that as a base without any worries.我刚刚将重大更改合并回主分支,因此您可以将其用作基础而无需担心。

Thank you for your efforts @sek-consulting I'll try to fix this. 😄

@declanchiu
Copy link
Contributor Author

@declanchiu please do :) I need to fix this for my current project and would appreciate it! 请这样做:)我需要为我当前的项目解决这个问题,我将不胜感激!

Ok, I'll try to fix this, thanks for using solid-ui! 😄

stefan-karger pushed a commit that referenced this issue Apr 3, 2024
* style: Fix missing folding animation for sheet component (#68)

* fix: missing modifications
@stefan-karger
Copy link
Owner

https://www.solid-ui.com/docs/components/sheet
The animation from the PR is merged and I also updated the component to be more inline with the shadcn version.

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

No branches or pull requests

3 participants