-
-
Notifications
You must be signed in to change notification settings - Fork 12
feat: allow animating to display: none; #25
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
base: main
Are you sure you want to change the base?
Conversation
|
Is there any documentation regarding browser support for using |
|
Other than browser support, I think this is a great idea. If you can get it to work for |
https://caniuse.com/?search=allow-discrete
Actually, after reducing it to simple plain css, I believe I have it working. I was probably messing up somewhere with setting it up. I'll push another commit with what should work based on the example below. |
|
👆 Correcting my former misunderstanding I think this will allow animating in to the authored css display from a I also renamed it from |
|
Hey there! Been an intense three weeks for me but this is still open, so I decided to take a look again. One thing I don't like is the |
|
Hi both of you |
Hello! I don't have time at the moment to check it out, but if their implementation works with tw-animate-css then let's go with that. I'm guessing you will be able to use it within custom animation @Keyframes also? |
display: none;https://www.youtube.com/watch?v=vmDEHAzj2XEdisplay: none, I am not sure how to animate back to it's original display. I have tried usinginitial,inherit,unset, andrevertbut they don't reset to the display set for the animated element by the css author, so it's a one way animation at the moment. We can't assumedisplay: block;, it could be using flex, grid, or any other displayhidden-outsincehiddenis Tailwind'sdisplay: none;class.