-
Notifications
You must be signed in to change notification settings - Fork 27
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
Upgrade Sass #1995
base: master
Are you sure you want to change the base?
Upgrade Sass #1995
Conversation
@@ -79,7 +79,7 @@ | |||
"json-server": "0.17.4", | |||
"nodemon": "3.1.7", | |||
"nyc": "17.1.0", | |||
"sass": "1.77.8", | |||
"sass-embedded": "1.80.6", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://www.npmjs.com/package/sass-embedded
This package is an alternative to the
sass
package. It supports the same JS
API assass
and is maintained by the same team, but where thesass
package
is pure JavaScript,sass-embedded
is instead a JavaScript wrapper around a
native Dart executable.
@@ -15,8 +15,7 @@ | |||
* along with this program. If not, see <http://www.gnu.org/licenses/>. | |||
*/ | |||
|
|||
@use '../settings.scss'; | |||
@import '@lumino/default-theme/style/index.css'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to JS import, as this seems to be falling back to CSS @import
instead of Sass @import
, and was no longer working as desired
import '@lumino/default-theme/style' |
Supersedes #1975, fixes the deprecation warnings it brings
vuetifyjs/vuetify#20489 (comment)
Check List
CONTRIBUTING.md
and added my name as a Code Contributor.?.?.x
branch.