Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix transitions on v-show cancelled on next frame (fix #7390) #7391

Merged

Conversation

Akryum
Copy link
Member

@Akryum Akryum commented Jan 5, 2018

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Guillaume Chau added 2 commits January 5, 2018 18:03
Add two tests (enter and leave) that cancels the transitions on v-show
When a transition is cancelled before the next frame, the `v-enter-to` or `v-leave-to` class was
still added to the element.

fix vuejs#7390
@Akryum Akryum requested a review from yyx990803 January 5, 2018 17:11
@Akryum Akryum self-assigned this Jan 5, 2018
@Akryum Akryum added the 2.x label Jan 5, 2018
@yyx990803 yyx990803 merged commit 5191f13 into vuejs:dev Jan 5, 2018
@Akryum Akryum deleted the 7390-transition-prevent-to-class-if-cancelled branch January 6, 2018 15:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants