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

fix(Suspense): fallback should work with transition #3968

Merged
merged 4 commits into from
Jun 21, 2021

Conversation

edison1105
Copy link
Member

@edison1105 edison1105 commented Jun 17, 2021

fix #3963

Copy link
Member

@posva posva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a test?

@edison1105
Copy link
Member Author

Can you add a test?

yes, too busy. I will do it later.

@edison1105 edison1105 requested a review from posva June 17, 2021 14:27
@edison1105
Copy link
Member Author

@posva
the test case has done. feel free to review.
time to go to bed.

await click('#toggleBtn')
await timeout(500)
expect(await html('#container')).toBe('<div class="">Loading...</div>')
await transitionFinish(1500)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using hard coded test durations like this.

See 740bcd1

@yyx990803 yyx990803 merged commit 43e2a72 into vuejs:master Jun 21, 2021
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.

Suspense emits fallback but does not displays the slot when using a transition
3 participants