Skip to content

Modernize SCSS setup#2764

Open
pichfl wants to merge 1 commit into
mainfrom
pichfl/sass
Open

Modernize SCSS setup#2764
pichfl wants to merge 1 commit into
mainfrom
pichfl/sass

Conversation

@pichfl

@pichfl pichfl commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Remove reliance on SCSS/SASS, replacing SCSS variables with native custom properties.

This should make it easier to drop SCSS and move to plain modern CSS and simpler bundling once we move the build chain to Vite.

This also moves importing the normalize.css package from ember-cli-build.js straight into SCSS, slowly chipping away at the vendor imports.

@pichfl pichfl added the internal label Jun 1, 2026
- Convert @import to meta.load-css to resolve warnings
- Convert most of the SCSS code to plain CSS to make remove SASS easier
"left" $nav-main-item-height - 6px
"right" $nav-main-item-height - 6px;
"left" calc(var(--nav-main-item-height) - 6px)
"right" var(--nav-main-item-height) - 6px;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing a calc() here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants