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

main.sass updated migration #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ashworks1706
Copy link

PR: Migrate Sass @import to @use

Description

This PR updates our Sass files to use the new @use rule instead of the deprecated @import rule. This change is necessary to prepare for future Sass versions and to leverage the benefits of the new module system.

Changes Made

  • Updated src/styles/main.sass to use @use instead of @import
  • Adjusted Vite configuration to use the modern Sass API
  • Updated dependencies to ensure compatibility with the new Sass syntax

Benefits

  • Future-proofs our Sass code for upcoming Sass versions
  • Improves modularity and reduces potential naming conflicts
  • Enhances performance by only importing what's needed

Testing

  • Verified that all styles are applied correctly after the migration
  • Ensured that the build process completes without errors
  • Checked that the application's appearance and functionality remain unchanged

Additional Notes

  • This change may require updates to how we reference variables, mixins, and functions from imported files. Instead of global access, we now need to use namespaced access (e.g., player.$variable instead of just $variable).
  • Developers should review the Sass documentation on @use to understand the new syntax and its implications.

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