Skip to content

fix(cdk/portal): insert DOM portal in right location - #34055

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:34025/portal-location
Nov 4, 2026
Merged

crisbeto merged 1 commit into
angular:mainfrom
crisbeto:34025/portal-location

Conversation

@crisbeto

@crisbeto crisbeto commented Nov 3, 2026

Copy link
Copy Markdown
Member

Fixes that if a DOM portal is attached to an ng-template, we were inserting the content into the end of the parent, rather than the portal outlet location.

Fixes #34025.

Fixes that if a DOM portal is attached to an `ng-template`, we were inserting the content into the end of the parent, rather than the portal outlet location.

Fixes angular#34025.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Nov 3, 2026
@pullapprove
pullapprove Bot requested review from ok7sai and tjshiu November 3, 2026 14:07
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Nov 4, 2026
@crisbeto
crisbeto removed the request for review from tjshiu November 4, 2026 08:36
@crisbeto
crisbeto merged commit e59fce8 into angular:main Nov 4, 2026
34 of 36 checks passed
@crisbeto

crisbeto commented Nov 4, 2026

Copy link
Copy Markdown
Member Author

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

crisbeto added a commit that referenced this pull request Nov 4, 2026
Fixes that if a DOM portal is attached to an `ng-template`, we were inserting the content into the end of the parent, rather than the portal outlet location.

Fixes #34025.

(cherry picked from commit e59fce8)
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/portal target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(cdk/portal): DOM node inserted at the wrong place

2 participants