Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(compiler-cli): ensure template style elements are preprocessed as inline styles #57429

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

Closed

Conversation

clydin
Copy link
Member

@clydin clydin commented Aug 16, 2024

Previously, style elements within a template were used directly and not provided to the optional transformation step that may be present on the resource host interface. This causes such styles to not be processed by the Angular CLI's stylesheet pipeline and could cause the styles to not work properly on all browsers. The style elements are now processed in the same manner as inline styles within a component's metadata. Link elements within a stylesheet were already being processed as styleUrls equivalent and there is no behavior change in that regard.

@clydin clydin added the target: major This PR is targeted for the next major release label Aug 16, 2024
@clydin clydin force-pushed the compiler/preprocess-style-elements branch from fa3e01d to a4b9e39 Compare August 16, 2024 19:03
@clydin clydin requested a review from JoostK August 19, 2024 12:08
@clydin clydin marked this pull request as ready for review August 19, 2024 12:08
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 19, 2024
@dylhunn dylhunn added the area: compiler Issues related to `ngc`, Angular's template compiler label Aug 19, 2024
@ngbot ngbot bot added this to the Backlog milestone Aug 19, 2024
…s inline styles

Previously, style elements within a template were used directly and not provided
to the optional transformation step that may be present on the resource host interface.
This causes such styles to not be processed by the Angular CLI's stylesheet pipeline
and could cause the styles to not work properly on all browsers. The style elements
are now processed in the same manner as inline styles within a component's metadata.
Link elements within a stylesheet were already being processed as `styleUrls`
equivalent and there is no behavior change in that regard.
@clydin clydin changed the title fix(compiler-cli): ensure template style elements are preprocessed as inline styles feat(compiler-cli): ensure template style elements are preprocessed as inline styles Aug 19, 2024
@clydin clydin force-pushed the compiler/preprocess-style-elements branch from a4b9e39 to 9312e07 Compare August 19, 2024 20:02
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Aug 19, 2024
Copy link
Member

@JoostK JoostK left a comment

Choose a reason for hiding this comment

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

LGTM with the suggestion to retarget as feat for just main.

@clydin clydin 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 Aug 19, 2024
@dylhunn
Copy link
Contributor

dylhunn commented Aug 20, 2024

This PR was merged into the repository by commit 9e87593.

The changes were merged into the following branches: main

@dylhunn dylhunn closed this in 9e87593 Aug 20, 2024
@clydin clydin deleted the compiler/preprocess-style-elements branch August 20, 2024 11:35
@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 Sep 20, 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: compiler Issues related to `ngc`, Angular's template compiler detected: feature PR contains a feature commit target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants