Skip to content

Optional queue transitions #8158

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

Closed
wants to merge 5 commits into from
Closed

Optional queue transitions #8158

wants to merge 5 commits into from

Conversation

Llorx
Copy link

@Llorx Llorx commented Oct 30, 2016

Right now ReactCSSTransitionGroup forces ending one transition to start the next one. This is not the behaviour everyone wants, so I added a transitionQueue property which is true by default, and if set to false will disable transitions queues, so will force ending the current transition and inmediatelly start the next one.

I added this because I wanted transitionLeave to start as soon an element is removed, instead of waiting for transitionEnter to finish.

@facebook-github-bot
Copy link

Thank you for your pull request. We require contributors to sign our Contributor License Agreement, and yours has expired.

Before we can review or merge your code, we need you to email [email protected] with your details so we can update your status.

@gaearon
Copy link
Collaborator

gaearon commented Oct 30, 2016

Hi, thanks for the PR! The problem is, we don't actively maintain TransitionGroup, and won't be able to review PRs adding new functionality to it. I encourage you to join the discussion in #8125 to figure out a way forward for it, and maybe help maintain it. Thanks, and sorry about closing the PR (we really appreciate your work!)

@gaearon gaearon closed this Oct 30, 2016
@cvle
Copy link

cvle commented Feb 1, 2017

Checkout this project as an alternative: https://github.com/wikiwi/react-css-transition

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.

4 participants