Skip to content

refactor(core): deprecate animations field on component interface #62895

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

refactor(core): deprecate animations field on component interface #62895

wants to merge 1 commit into from

Conversation

thePunderWoman
Copy link
Contributor

This deprecates the animations field on the component decorator in favor of animate.enter and leave.

DEPRECATED: @angular/animations

@thePunderWoman thePunderWoman requested a review from crisbeto July 30, 2025 09:49
@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime target: minor This PR is targeted for the next minor release labels Jul 30, 2025
@ngbot ngbot bot added this to the Backlog milestone Jul 30, 2025
@angular-robot angular-robot bot added the detected: deprecation PR contains a commit with a deprecation label Jul 30, 2025
@thePunderWoman
refactor(core): deprecate animations field on component interface
3a0f5f8 
This deprecates the animations field on the component decorator in favor of animate.enter and leave.

DEPRECATED: @angular/animations
@pullapprove pullapprove bot requested review from AndrewKushnir and atscott July 30, 2025 09:52
JeanMeche
JeanMeche approved these changes Jul 30, 2025
View reviewed changes
Copy link
Member

@JeanMeche JeanMeche left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

@pullapprove pullapprove bot requested a review from devversion July 30, 2025 09:54
crisbeto
crisbeto approved these changes Jul 30, 2025
View reviewed changes
Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

Reviewed-for: public-api

@thePunderWoman
Copy link
Contributor Author

This PR was merged into the repository by commit 320de4e.

The changes were merged into the following branches: main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JeanMeche JeanMeche JeanMeche approved these changes

@crisbeto crisbeto crisbeto approved these changes

Assignees
No one assigned
Labels
action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime detected: deprecation PR contains a commit with a deprecation target: minor This PR is targeted for the next minor release
Projects
None yet
Milestone
Backlog
Development

Successfully merging this pull request may close these issues.
3 participants
@thePunderWoman @JeanMeche @crisbeto