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

When using stagger, the initial delay set to 0 #202

Closed
wants to merge 1 commit into from
Closed

When using stagger, the initial delay set to 0 #202

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 23, 2014

When stagger is set on a sequence, the delay option is completely ignored. I think it should be added to the calculation for a more consistent behaviour.

When stagger is set on a sequence, the delay option is completely ignored. I think it should be added to the calculation for a more consistent behaviour.
@julianshapiro
Copy link
Owner

Were you trying this with a UI pack effect or your own custom sequence?

@ghost
Copy link
Author

ghost commented Jul 23, 2014

With the ui pack. I figured, looking at the code, it would apply to both. Want a demo exploring this more?

@julianshapiro
Copy link
Owner

Fixed. Please update Velocity and let me know if this issue persists. Thanks so much, @Guanche.

@ghost ghost deleted the patch-1 branch October 24, 2014 11:23
Rycochet pushed a commit that referenced this pull request Aug 3, 2020
Switch “animating” indicator class to “velocity-animating”. Closes #194.

Support for passing hex values into color properties. (Previously, you
could only pass in individual RGBA components.) Closes #179.

Fixes bug where slide functions wouldn’t reset elements’ height to
auto. Closes #183.

Support for getting border-color in Firefox. Closes #196.

Fixes inability to animate `y2` on SVG elements. Closes #199.

Allow passing in explicit `display: none` into an Out UI pack
transition. Closes #201.

Fixes vw/vh unit support for `translateX/Y`. Closes #181.

Fixes bug where `delay` couldn’t be used with `stagger`. Closes #202.
Thank you, @Guanche.

Fixes bug where original call’s easing type wouldn’t be re-used for a
`reverse` command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants