Skip to content

fix: shoud determine whether the component is react.Fragment #96

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zaleGZL
Copy link

@zaleGZL zaleGZL commented Feb 17, 2020

React.Fragment has only one key property, we need to add judgement, we cannot pass other properties to React.Fragment.

example:
image

@vercel
Copy link

vercel bot commented Feb 17, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

@coveralls
Copy link

coveralls commented Feb 17, 2020

Coverage Status

Coverage increased (+0.04%) to 82.816% when pulling af54caa on zaleGZL:fix_fragment_error into a1385aa on react-component:master.

@zaleGZL
Copy link
Author

zaleGZL commented Feb 17, 2020

@afc163 Can you look at this pr?

@zaleGZL zaleGZL closed this Feb 22, 2020
@afc163 afc163 reopened this Feb 22, 2020
@afc163
Copy link
Member

afc163 commented Feb 22, 2020

Could you please fix ci?

@zombieJ
Copy link
Member

zombieJ commented Feb 22, 2020

This fix just make Animate not throw error but also make Animation not work with React.Fragment. Could you provide more info like reproduce that why you need use Fragment with rc-animatie?

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