Skip to content

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Nov 4, 2025

Includes the following popover-related changes in the overlay:

refactor(cdk/overlay): tweak popover resets

Updates the popover resets based on recent tests.

refactor(cdk/overlay): keep global overlays inside the container

Keeps global overlays inside the overlay container since it can be breaking for users to move them out.

refactor(cdk/overlay): allow connected overlays to stay in the overlay container

Adds an API that allows connected overlays to decide if they should be inline or in the overlay container when they're rendered out as popovers.

refactor(material/tooltip): render tooltips as popovers

Tests out rendering tooltips as popovers inside the overlay container.

Updates the popover resets based on recent tests.
@crisbeto crisbeto added merge: preserve commits When the PR is merged, a rebase and merge should be performed target: rc This PR is targeted for the next release-candidate labels Nov 4, 2025
@crisbeto crisbeto marked this pull request as ready for review November 4, 2025 13:36
@crisbeto crisbeto requested a review from a team as a code owner November 4, 2025 13:36
@crisbeto crisbeto requested review from andrewseguin and ok7sai and removed request for a team November 4, 2025 13:36
Keeps global overlays inside the overlay container since it can be breaking for users to move them out.
@crisbeto crisbeto force-pushed the popover-tweaks branch 2 times, most recently from e4d7c06 to 45036ce Compare November 4, 2025 19:44
@crisbeto crisbeto removed the request for review from ok7sai November 4, 2025 19:45
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Nov 4, 2025
…y container

Adds an API that allows connected overlays to decide if they should be inline or in the overlay container when they're rendered out as popovers.
Tests out rendering tooltips as popovers inside the overlay container.
@crisbeto crisbeto merged commit 31e9100 into angular:main Nov 4, 2025
20 of 22 checks passed
@crisbeto
Copy link
Member Author

crisbeto commented Nov 4, 2025

This PR was merged into the repository. The changes were merged into the following branches:

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: cdk/overlay area: material/tooltip merge: preserve commits When the PR is merged, a rebase and merge should be performed target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants