Skip to content

fix(material/select): switch away from animations module #30363

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

Merged
merged 3 commits into from
Jan 23, 2025

Conversation

crisbeto
Copy link
Member

Reworks the select so it doesn't depend on the animations module.

…verlay directive

Adds methods on the `CdkConnectedOverlay` to allow for manual attachment/detachment, instead of going through change detection.
Reworks the select so it doesn't depend on the animations module.
@crisbeto crisbeto added target: minor This PR is targeted for the next minor release merge: preserve commits When the PR is merged, a rebase and merge should be performed labels Jan 21, 2025
@crisbeto crisbeto requested a review from a team as a code owner January 21, 2025 22:05
@crisbeto crisbeto requested review from andrewseguin and wagnermaciel and removed request for a team January 21, 2025 22:05
@crisbeto crisbeto force-pushed the select-no-animations branch from 334fc91 to e09370d Compare January 23, 2025 10:06
@crisbeto crisbeto force-pushed the select-no-animations branch from e09370d to 93097f6 Compare January 23, 2025 12:54
@crisbeto crisbeto removed the request for review from wagnermaciel January 23, 2025 17:13
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jan 23, 2025
@crisbeto crisbeto merged commit b7c377b into angular:main Jan 23, 2025
22 checks passed
crisbeto added a commit to crisbeto/material2 that referenced this pull request Jan 28, 2025
Fixes a regression after angular#30363 where the select panel throws an error if `open` and `close` are called before Angular has had a chance to resolve the `panel`.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Jan 28, 2025
Fixes a regression after angular#30363 where the select panel throws an error if `open` and `close` are called before Angular has had a chance to resolve the `panel`.
crisbeto added a commit that referenced this pull request Jan 28, 2025
)

Fixes a regression after #30363 where the select panel throws an error if `open` and `close` are called before Angular has had a chance to resolve the `panel`.
@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 Feb 23, 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: cdk/overlay area: material/select merge: preserve commits When the PR is merged, a rebase and merge should be performed 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