Skip to content

fix(angular): use SASS indented syntax in nx-welcome component when style is sass#34510

Merged
Coly010 merged 1 commit intomasterfrom
angular-css-sass
Feb 23, 2026
Merged

fix(angular): use SASS indented syntax in nx-welcome component when style is sass#34510
Coly010 merged 1 commit intomasterfrom
angular-css-sass

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Feb 19, 2026

The nx-welcome component inline styles were always using CSS/SCSS syntax
(with braces and semicolons) regardless of the selected style option.
When --style=sass is chosen, the component now correctly uses SASS
indented syntax (no braces or semicolons) matching the expected
behavior for the .sass file format.

Fixes #33489

@Coly010 Coly010 requested a review from a team as a code owner February 19, 2026 12:54
@Coly010 Coly010 requested a review from jaysoo February 19, 2026 12:54
@Coly010 Coly010 self-assigned this Feb 19, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 19, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit cc3e166
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/699c22d146d89800084cd3f2
😎 Deploy Preview https://deploy-preview-34510--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 19, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit cc3e166
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/699c22d1c1c89d0008298f46
😎 Deploy Preview https://deploy-preview-34510--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud bot commented Feb 19, 2026

View your CI Pipeline Execution ↗ for commit cc3e166

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 27m 12s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 3m 9s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 7s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-23 10:36:05 UTC

…tyle is sass

The nx-welcome component inline styles were always using CSS/SCSS syntax
(with braces and semicolons) regardless of the selected style option.
When --style=sass is chosen, the component now correctly uses SASS
indented syntax (no braces or semicolons) matching the expected
behavior for the .sass file format.
@Coly010 Coly010 merged commit 6bcaa46 into master Feb 23, 2026
24 of 25 checks passed
@Coly010 Coly010 deleted the angular-css-sass branch February 23, 2026 11:50
FrozenPandaz pushed a commit that referenced this pull request Feb 26, 2026
…tyle is sass (#34510)

The nx-welcome component inline styles were always using CSS/SCSS syntax
(with braces and semicolons) regardless of the selected style option.
When --style=sass is chosen, the component now correctly uses SASS
indented syntax (no braces or semicolons) matching the expected
behavior for the .sass file format.

Fixes #33489

(cherry picked from commit 6bcaa46)
FrozenPandaz pushed a commit that referenced this pull request Feb 26, 2026
…tyle is sass (#34510)

The nx-welcome component inline styles were always using CSS/SCSS syntax
(with braces and semicolons) regardless of the selected style option.
When --style=sass is chosen, the component now correctly uses SASS
indented syntax (no braces or semicolons) matching the expected
behavior for the .sass file format.

Fixes #33489

(cherry picked from commit 6bcaa46)
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 1, 2026

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace default template's CSS with SASS if new angular project is setup with SASS

2 participants