Skip to content

fix(@angular-devkit/build-angular): check namespaced Sass variables when rebasing URLs#26577

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:sass/rebase-namespaced-variables
Dec 5, 2023
Merged

fix(@angular-devkit/build-angular): check namespaced Sass variables when rebasing URLs#26577
clydin merged 1 commit intoangular:mainfrom
clydin:sass/rebase-namespaced-variables

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 4, 2023

The @use Sass directive allows Sass variables to be accessed via a namespace. This was previously not checked when performing URL path rebasing on imported Sass stylesheets. These type of Sass variable references are now also ignored when attempting to rebase URL paths during loading. The final rebased URL now also does not add a leading relative prefix indicator (./) unless not already present.

Closes #26570

…hen rebasing URLs

The `@use` Sass directive allows Sass variables to be accessed via a namespace. This was
previously not checked when performing URL path rebasing on imported Sass stylesheets.
These type of Sass variable references are now also ignored when attempting to rebase URL
paths during loading. The final rebased URL now also does not add a leading relative prefix
indicator (`./`) unless not already present.
@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Dec 4, 2023
@clydin clydin requested a review from alan-agius4 December 4, 2023 16:40
@alan-agius4 alan-agius4 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 5, 2023
@clydin clydin merged commit 8e9b675 into angular:main Dec 5, 2023
@clydin clydin deleted the sass/rebase-namespaced-variables branch December 5, 2023 15:15
@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 Jan 5, 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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Third party library using an exported scss variable from another third party library is throwing issue while using application builder

2 participants