-
-
Notifications
You must be signed in to change notification settings - Fork 228
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
feature request: enable only the animations we need #153
Comments
I think this #118 is related |
It would be nice to have a bit more control over it and it's defaults. Eg.: I want to override just add animation and keep others as default. In theory we could modify the config to have: onAdd, onRemove, onRemain and if omitted it would pick the defaults for it. Probably solves two things at once, if user pass false to any of these, than it wouldn't animate that action. |
So is there any easy way to do this now? I'm also trying to disable the |
I ended up forking it on our private monorepo and added the callbacks I said earlier, most likely the pull request wasn't going to be accepted so I didn't even tried to make one. |
It would be great to be able to pass the animations we need, like:
My app uses
react-beautiful-dnd
and it doesn't work very well with 'remain', so in cases like this when we use other libraries with animations that might interfere with the behavior of the element, this function will be really useful.The text was updated successfully, but these errors were encountered: