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.

Sign up for GitHub

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

Jump to bottom

Replace Sass @import usage with @forward #105

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

Replace Sass @import usage with @forward #105

wants to merge 1 commit into from

Commits on Feb 14, 2023

  1. Replace SASS @import usage with @forward 

    The use of @import is discouraged. Utilize the @use rule instead.

See: <https://sass-lang.com/documentation/at-rules/import>
"The Sass team discourages the continued use of the @import rule. Sass will
gradually phase it out over the next few years, and eventually remove it from
the language entirely. Prefer the @use rule instead."

Changelog:

* Removed extraeneous $color-text comments from scss/css files
* Implemented @use over @import
* scss/_main.scss defaults to the Sakura base theme
  * In turn, scss/sakura.scss does not specify overrides
* Updated test.html to include missing themes
  * Also added padding to theme switcher labels
* Updated README 'earthy' theme example
* Updated package-lock.json version 3 (latest stable node/npm)
  * Reference versions: node 18.14.0 (LTS) / npm 9.3.1
    @rmkane
    rmkane committed Feb 14, 2023
    Configuration menu
    Copy the full SHA
    42d8455 View commit details
    Browse the repository at this point in the history