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

Breaks CSSTransitionGroup enter animations #165

Closed
anthonator opened this issue May 4, 2015 · 15 comments · Fixed by #4625
Closed

Breaks CSSTransitionGroup enter animations #165

anthonator opened this issue May 4, 2015 · 15 comments · Fixed by #4625
Labels
issue/bug-confirmed Issues about a bug that has been confirmed by a maintainer issue/reviewed Issue has recently been reviewed (mid-2020)

Comments

@anthonator
Copy link

When using react-select I'm seeing enter animations breaking. They simply don't animate. The leave animations continue to work. If I remove the Select component from my view the animations start working again.

@VictorGarcia
Copy link

Any updates on this? I'm experiencing the same issue...

@B-Reif
Copy link

B-Reif commented Jul 20, 2017

I'm having this issue too (almost a year later). I was able to restore the animation by setting searchable={false}.

Maybe the search input box is slowing down render times too much? Maybe this is related somehow to #733.

@mmellado
Copy link

mmellado commented Jan 2, 2018

Bump, experiencing this as well

@jjmr
Copy link

jjmr commented Apr 25, 2018

still happening after 3 years...any clue?

@jossmac
Copy link
Collaborator

jossmac commented Aug 29, 2018

Is this still an issue in v2?

@jossmac jossmac added the v1 label Aug 29, 2018
@isakfagerlund
Copy link

Is this still an issue in v2?

This still happens in 2.1.0

@jossmac
Copy link
Collaborator

jossmac commented Oct 8, 2018

Could someone please provide a reproduction in CodeSandbox?

@jossmac jossmac added issue/bug-unconfirmed Issues that describe a bug that hasn't been confirmed by a maintainer yet and removed v1 labels Oct 8, 2018
@bryant-sharethis
Copy link

same issue here

@gwyneplaine
Copy link
Collaborator

please feel free to reopen this with a codesandbox reproduction.

@foxinni
Copy link

foxinni commented Mar 15, 2019

It seems like isSearchable={false} for V2 fixes the CSS transition.

@OwenFlood
Copy link

I set up a quick codesandbox with both the searchable enabled and disabled to show the animation working/not working.

https://codesandbox.io/s/affectionate-rhodes-qir59

@twheys
Copy link

twheys commented Sep 17, 2019

@gwyneplaine Could you please reopen this ticket given that @OwenFlood has posted a codesandbox link? This bug is still a thing.

@saltnpixels
Copy link

I made a useEffect that change the isSearchable to true after 1 millisecond....

const [isSearchable, setIsSearchable] = useState(false)
setTimeout(() => {
         setIsSearchable(true)
      }, 1)

But ultimately this needs a real fix....

@ebonow ebonow added the issue/bug-confirmed Issues about a bug that has been confirmed by a maintainer label Dec 1, 2020
@ebonow
Copy link
Collaborator

ebonow commented Dec 1, 2020

I set up a quick codesandbox with both the searchable enabled and disabled to show the animation working/not working.

https://codesandbox.io/s/affectionate-rhodes-qir59

I tested the link as provided, confirmed that this is indeed still an issue, and will reopen this.

@marianayap has been actively volunteering for issues and if she or anyone else would like to work on a fix and PR, then we can add this to the queue of PRs for release.

As an update to everyone, you can read more here, but...
TLDR: We've added collaborators and currently choosing PR candidates for v3.2.

@ebonow ebonow reopened this Dec 1, 2020
@ebonow ebonow added issue/reviewed Issue has recently been reviewed (mid-2020) and removed issue/bug-unconfirmed Issues that describe a bug that hasn't been confirmed by a maintainer yet labels Dec 2, 2020
@ebonow
Copy link
Collaborator

ebonow commented Dec 3, 2020

As expected, this appears to be an issue related to the AutosizeInput used in the Input component.

Here is a fork of the codesandbox example demonstrating the same behavior of AutosizeInput vs input here

@ebonow ebonow linked a pull request Jun 4, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/bug-confirmed Issues about a bug that has been confirmed by a maintainer issue/reviewed Issue has recently been reviewed (mid-2020)
Projects
None yet
Development

Successfully merging a pull request may close this issue.