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(material/autocomplete): switch away from animations module #30356

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

crisbeto
Copy link
Member

Switches the autocomplete away from the animations module. The animations are implemented using CSS instead.

Switches the autocomplete away from the animations module. The animations are implemented using CSS instead.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Jan 20, 2025
@crisbeto crisbeto requested a review from a team as a code owner January 20, 2025 13:28
@crisbeto crisbeto requested review from amysorto and mmalerba and removed request for a team January 20, 2025 13:28
}
}

// Note: the autocomplete intentionally only implements an enter animation.
Copy link
Member Author

Choose a reason for hiding this comment

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

I decided to do this, because the autocomplete is quite jumpy on selection at the moment and implementing the exit animation requires some extra code.

@crisbeto crisbeto requested a review from andrewseguin January 20, 2025 13:29
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Jan 21, 2025
@crisbeto crisbeto merged commit 013fe04 into angular:main Jan 21, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: material/autocomplete target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants