Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Nov 18, 2024

The component stylesheet HMR functionality requires build-time analysis of each component by the AOT compiler to provide the needed information to identify initial styles and detect individual changes to each style. Part of the style HMR rebuild logic was unintentionally enabled in JIT mode. The initial load of the application operated correctly but subsequent changes to file-based stylesheets were delayed by one rebuild cycle. To avoid this misalignment, all component stylesheet HMR functionality is now disabled when in JIT mode.

The component stylesheet HMR functionality requires build-time analysis
of each component by the AOT compiler to provide the needed information
to identify initial styles and detect individual changes to each style.
Part of the style HMR rebuild logic was unintentionally enabled in JIT
mode. The initial load of the application operated correctly but subsequent
changes to file-based stylesheets were delayed by one rebuild cycle. To
avoid this misalignment, all component stylesheet HMR functionality is
now disabled when in JIT mode.
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Nov 18, 2024
@clydin clydin requested a review from dgp1130 November 18, 2024 15:17
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Nov 18, 2024
@dgp1130 dgp1130 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 Nov 18, 2024
@dgp1130 dgp1130 merged commit 9921271 into angular:main Nov 18, 2024
33 of 34 checks passed
@dgp1130
Copy link
Collaborator

dgp1130 commented Nov 18, 2024

The changes were merged into the following branches: main, 19.0.x

@dgp1130 dgp1130 added this to the v19 Candidates milestone Nov 18, 2024
@clydin clydin deleted the hmr/disable-in-jit branch November 18, 2024 18:08
@clydin clydin linked an issue Nov 18, 2024 that may be closed by this pull request
1 task
@angular-automatic-lock-bot
Copy link

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 Dec 19, 2024
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: @angular/build target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HMR with external component styles + jit mode shows previous update on save
2 participants