Skip to content
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

Remove SCSS preprocessor #39

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Remove SCSS preprocessor #39

wants to merge 6 commits into from

Conversation

devenini
Copy link
Member

@devenini devenini commented Jan 8, 2025

Summary

This PR removes SCSS from the project and switches to using Tailwind CSS and PostCSS. All variables are now handled in the Tailwind config, and PostCSS plugins are used to cover SCSS-like features such as nesting and custom needs. SCSS was also blocking some Tailwind features from being used, so this change makes the setup simpler and more effective.

Another reason for this change is that SCSS keeps updating and deprecating features, which adds extra work to maintain. Since we don’t really need SCSS anymore, removing it makes the project easier to manage.

Changes

  • Removed SCSS files and dependencies.
  • Updated styles to work with Tailwind CSS and PostCSS.
  • Set up PostCSS plugins for nesting and other project-specific features.
  • Cleaned up the build and tooling setup.

Benefits

  • Makes the project simpler by reducing dependencies.
  • Takes full advantage of Tailwind CSS features.
  • Cuts down on maintenance work by removing SCSS and its updates.
  • PostCSS plugins allow flexibility for any custom styling needs.

For more details on our PostCSS setup, check out the Front-End Helpers repository.

Copy link

vercel bot commented Jan 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
astro-boilerplate ✅ Ready (Inspect) Visit Preview Jan 8, 2025 8:18pm

README.md Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
tailwind.config.ts Show resolved Hide resolved
@devenini
Copy link
Member Author

devenini commented Jan 8, 2025

Forgot to mention that it will also fix #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants