-
Notifications
You must be signed in to change notification settings - Fork 26k
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
Project the root node of empty blocks #53620
Conversation
Expands the workaround from angular#52414 to allow for the root nodes of `@empty` blocks to be content projected. Fixes angular#53570.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - this was fast!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mentioned you were worried the template pipeline part was messy, but IMO it's not that messy! Just the one thing you already pointed out, which is a known refactoring task :)
Updates the template pipeline to allow for the root node of an `@empty` block to be content projected.
9ea6f3d
to
78a855e
Compare
This PR was merged into the repository by commit 478d622. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Expands the workaround from angular#52414 to allow for the root nodes of `@empty` blocks to be content projected. Fixes angular#53570. PR Close angular#53620
…gular#53620) Updates the template pipeline to allow for the root node of an `@empty` block to be content projected. PR Close angular#53620
Expands the workaround from angular#52414 to allow for the root nodes of `@empty` blocks to be content projected. Fixes angular#53570. PR Close angular#53620
…gular#53620) Updates the template pipeline to allow for the root node of an `@empty` block to be content projected. PR Close angular#53620
Expands the workaround from angular#52414 to allow for the root nodes of `@empty` blocks to be content projected. Fixes angular#53570. PR Close angular#53620
…gular#53620) Updates the template pipeline to allow for the root node of an `@empty` block to be content projected. PR Close angular#53620
Expands the workaround from angular#52414 to allow for the root nodes of `@empty` blocks to be content projected. Fixes angular#53570. PR Close angular#53620
…gular#53620) Updates the template pipeline to allow for the root node of an `@empty` block to be content projected. PR Close angular#53620
Includes a couple of commits that allow for the root node of an
@empty
block to be content projected.fix(compiler): project empty block root node
Expands the workaround from #52414 to allow for the root nodes of
@empty
blocks to be content projected.fix(compiler): project empty block root node in template pipeline
Updates the template pipeline to allow for the root node of an
@empty
block to be content projected.Fixes #53570.