Skip to content

fix(core): prevent animation events from being cleaned up on destroy#63414

Closed
thePunderWoman wants to merge 1 commit intoangular:mainfrom
thePunderWoman:animation-end
Closed

fix(core): prevent animation events from being cleaned up on destroy#63414
thePunderWoman wants to merge 1 commit intoangular:mainfrom
thePunderWoman:animation-end

Conversation

@thePunderWoman
Copy link
Contributor

This will allow manually subscribed animation events to still fire when using animate.leave. Otherwise they were being cleaned up before the animations happened.

fixes: #63391

@thePunderWoman thePunderWoman added action: review The PR is still awaiting reviews from at least one requested reviewer area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release labels Aug 27, 2025
@ngbot ngbot bot added this to the Backlog milestone Aug 27, 2025
@thePunderWoman thePunderWoman added action: discuss and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 27, 2025
This will allow manually subscribed animation events to still fire when using `animate.leave`. Otherwise they were being cleaned up before the animations happened.

fixes: angular#63391
@thePunderWoman thePunderWoman removed the request for review from crisbeto September 9, 2025 23:46
@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker and removed action: discuss labels Sep 9, 2025
atscott pushed a commit that referenced this pull request Sep 10, 2025
…63414)

This will allow manually subscribed animation events to still fire when using `animate.leave`. Otherwise they were being cleaned up before the animations happened.

fixes: #63391

PR Close #63414
@atscott
Copy link
Contributor

atscott commented Sep 10, 2025

This PR was merged into the repository by commit 3d6eeab.

The changes were merged into the following branches: main, 20.3.x

@atscott atscott closed this in 3d6eeab Sep 10, 2025
@thePunderWoman thePunderWoman deleted the animation-end branch September 12, 2025 16:37
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Oct 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

animate.leave does not emit animation events outside of the instructions

3 participants