Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Transition is "appearing" by default and the documentation says that it is false as default

See original GitHub issue

The issue is present in the latest release.
I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When using the component Transition, the content “appears” instead of having the default behavior of not appearing on the first mount.

Expected Behavior 🤔

The component should not appear on the first mount, as stated on the documentation.

Steps to Reproduce 🕹

Steps:

Use any Transition component without “appear” property.

https://codesandbox.io/s/nice-hooks-fpi84?file=/src/Demo.js

Context 🔦

Your Environment 🌎

Tech	Version
Material-UI	v4.11.0
React	16.13.1
Browser	Google Chrome 84.0.4147.125 (Official Build) (64-bit)
TypeScript	3.9.7

Issue Analytics

State:
Created 3 years ago
Comments:10 (9 by maintainers)

Top GitHub Comments

2reactions

eps1loncommented, Aug 14, 2020

Maybe we could have a new prop disableAppear to have the default as false and avoid the change of behavior.

I think we can make an argument the documenting appear as a default implies that <Grow appear={undefined} /> leads to <Transition appear={true} />. Let’s just call it a bug (appear should default to true; previously omitting appear was treated differently than setting appear to undefined) and schedule it for v5 to reduce risk of unwanted breakage.

1reaction

GuilleDFcommented, Oct 23, 2020

Hi! I can take this if it’s okay! 😄