-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
UI Pack: Height animation with dynamic child elements #110
Comments
I think I'm going to code this directly into the UI pack. Stay tuned. |
Awesome, thanks! I will then write a Knockout JS custom binding for Velocity UI. |
Cool! Will wait -Sri On Thu, Jul 3, 2014 at 8:36 PM, Julian Shapiro [email protected]
|
Lol. I just coded this feature into my local copy of the UI pack. It's cool as shit. Problem is, it's fundamentally not very performant. Triggers a ton of unavoidable layout thrashing. Wouldn't have much of a visible impact in most/low-stress situations, but still against my wishes. Will chew on this. |
👍 can't wait to see it in action! |
OK. So this is finally integrated. But I haven't documented it yet. I'm going to create a new thread tonight or tomorrow morning in which I'll explain how to use it plus ask you guys for feedback. Stay tuned. In the meantime, update Velocity and the UI pack. |
Damn. There's a bug. Will fix and follow up tomorrow. |
Hi,
I've made a small demo to illustrate what I would like to achieve:
http://www.marcselman.com/velocity/smooth-space.html
Check the 'In' transitions with a stagger of 300.
As you can see the container has an height of 'auto' and the required space for the elements is claimed in a smooth animation.
It's a simple dirty hack in http://www.marcselman.com/velocity/jquery.velocity.js around lines 1390 to 1425.
Would it be possible to add a option to Velocity (animateParentHeight or something) which resizes the parent when animating elements in or out of view?
I think it could be a unique and liked feature.
I don't really see any way I could do this without changing the source code.
The text was updated successfully, but these errors were encountered: