Skip to content

refactor(core): track dirtiness bits in ApplicationRef - #56934

Closed
alxhub wants to merge 1 commit into
angular:mainfrom
alxhub:appref-dirty
Closed

alxhub wants to merge 1 commit into
angular:mainfrom
alxhub:appref-dirty

Conversation

@alxhub

@alxhub alxhub commented Jul 10, 2024

Copy link
Copy Markdown
Member

Previously the zoneless scheduler had a concept of whether views needed to be refreshed or not, based on the notification type that was received. It tracked this information as a boolean.

This commit refactors things to track dirtiness in ApplicationRef itself, as a dirtyFlags field with bits corresponding to either view tree dirtiness or after-render hooks.

@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Jul 10, 2024
@ngbot ngbot Bot added this to the Backlog milestone Jul 10, 2024
@alxhub
alxhub requested a review from atscott July 10, 2024 19:15
@atscott atscott added the target: patch This PR is targeted for the next patch release label Jul 10, 2024
@alxhub
alxhub force-pushed the appref-dirty branch 2 times, most recently from 0a8da84 to 111369c Compare July 17, 2024 15:31
@alxhub alxhub added requires: TGP This PR requires a passing TGP before merging is allowed and removed state: blocked labels Jul 17, 2024
@pullapprove pullapprove Bot removed the requires: TGP This PR requires a passing TGP before merging is allowed label Jul 17, 2024
@alxhub
alxhub force-pushed the appref-dirty branch 2 times, most recently from 2f1908a to cfb7bfa Compare July 17, 2024 15:48
Previously the zoneless scheduler had a concept of whether views needed to
be refreshed or not, based on the notification type that was received. It
tracked this information as a boolean.

This commit refactors things to track dirtiness in `ApplicationRef` itself,
as a `dirtyFlags` field with bits corresponding to either view tree
dirtiness or after-render hooks.
@alxhub

alxhub commented Aug 19, 2024

Copy link
Copy Markdown
Member Author

Closing in favor of #57453

@alxhub alxhub closed this Aug 19, 2024
@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 Sep 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants