Skip to content

fix(core): support hydration for cases when content is re-projected using ng-template#53304

Closed
AndrewKushnir wants to merge 1 commit intoangular:mainfrom
AndrewKushnir:hydration_projection_fix_1
Closed

fix(core): support hydration for cases when content is re-projected using ng-template#53304
AndrewKushnir wants to merge 1 commit intoangular:mainfrom
AndrewKushnir:hydration_projection_fix_1

Conversation

@AndrewKushnir
Copy link
Copy Markdown
Contributor

This commit fixes an issue with hydration, which happens when a content is projected in a certain way, leaving host elements non-projected, but the child content projected.

The fix is to detect such situations and add extra annotations to help runtime logic locate those elements at the right locations.

Resolves #53276.

PR Type

What kind of change does this PR introduce?

  • Bugfix

Does this PR introduce a breaking change?

  • Yes
  • No

…sing ng-template

This commit fixes an issue with hydration, which happens when a content is projected in a certain way, leaving host elements non-projected, but the child content projected.

The fix is to detect such situations and add extra annotations to help runtime logic locate those elements at the right locations.

Resolves angular#53276.
@AndrewKushnir AndrewKushnir added action: review The PR is still awaiting reviews from at least one requested reviewer area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release core: hydration labels Dec 1, 2023
@ngbot ngbot bot added this to the Backlog milestone Dec 1, 2023
Copy link
Copy Markdown
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

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

Looks great!

@jessicajaniuk jessicajaniuk added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 1, 2023
@dylhunn
Copy link
Copy Markdown
Contributor

dylhunn commented Dec 1, 2023

This PR was merged into the repository by commit 4b23221.

@dylhunn dylhunn closed this in 4b23221 Dec 1, 2023
dylhunn pushed a commit that referenced this pull request Dec 1, 2023
…sing ng-template (#53304)

This commit fixes an issue with hydration, which happens when a content is projected in a certain way, leaving host elements non-projected, but the child content projected.

The fix is to detect such situations and add extra annotations to help runtime logic locate those elements at the right locations.

Resolves #53276.

PR Close #53304
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Jan 1, 2024
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
…sing ng-template (angular#53304)

This commit fixes an issue with hydration, which happens when a content is projected in a certain way, leaving host elements non-projected, but the child content projected.

The fix is to detect such situations and add extra annotations to help runtime logic locate those elements at the right locations.

Resolves angular#53276.

PR Close angular#53304
rlmestre pushed a commit to rlmestre/angular that referenced this pull request Jan 26, 2024
…sing ng-template (angular#53304)

This commit fixes an issue with hydration, which happens when a content is projected in a certain way, leaving host elements non-projected, but the child content projected.

The fix is to detect such situations and add extra annotations to help runtime logic locate those elements at the right locations.

Resolves angular#53276.

PR Close angular#53304
amilamen pushed a commit to amilamen/angular that referenced this pull request Jan 26, 2024
…sing ng-template (angular#53304)

This commit fixes an issue with hydration, which happens when a content is projected in a certain way, leaving host elements non-projected, but the child content projected.

The fix is to detect such situations and add extra annotations to help runtime logic locate those elements at the right locations.

Resolves angular#53276.

PR Close angular#53304
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: core Issues related to the framework runtime core: hydration target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hydration error when component's content is re-projected using ng-template

3 participants