Skip to content

fix(overlay): infinite loop when used together with zone-patch-rxjs - #13081

Merged
jelbourn merged 1 commit into
angular:masterfrom
crisbeto:12686/overlay-zone-infinite-loop
Sep 14, 2018
Merged

fix(overlay): infinite loop when used together with zone-patch-rxjs#13081
jelbourn merged 1 commit into
angular:masterfrom
crisbeto:12686/overlay-zone-infinite-loop

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Fixes the overlay causing the browser to go into an infinite loop if the consumer is using zone-patch-rxjs.

Fixes#12686.

Fixes the overlay causing the browser to go into an infinite loop if the consumer is using `zone-patch-rxjs`.
Fixesangular#12686.
@crisbetocrisbeto added P2 The issue is important to a large percentage of users, with a workaround target: patch This PR is targeted for the next patch release labels Sep 11, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Sep 11, 2018
@crisbeto

crisbeto commented Sep 11, 2018

Copy link
Copy Markdown
MemberAuthor

Marking as P2, because the overlay touches a lot of components and these changes are fixing a regression.

@jelbournjelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jelbournjelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Sep 11, 2018
@jelbourn
jelbourn merged commit c2e502c into angular:masterSep 14, 2018
subscription.unsubscribe();
}
});
// Keeping the host element in DOM the can cause scroll jank, because it still gets

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

typo: "in DOM the" -> "in the DOM"

@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-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 10, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License AgreementP2The issue is important to a large percentage of users, with a workaroundtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Infinite loop when zone-patch-rxjs is used

5 participants

@crisbeto@petebacondarwin@jelbourn@googlebot@josephperrott