Bootstrap-sass v3.3.7 generates a "Using / for division is deprecated and will be removed in Dart Sass 2.0.0." warning when using laravel mix
realtebo opened this issue · 1 comments
I know v3.3.7 is a very old versione now.
But we are using it due to a long-life laravel project using a specific bought laravel admin theme.
Actually, laravel mix is using dart-sass to compile it. I don't know if it's related or not.
We have A LOT of warning (more than 30) about this
28 │ margin-bottom: ($line-height-computed / 2);
│ ^^^^^^^^^^^^^^^^^^^^^^^^^
╵
node_modules\bootstrap-sass\assets\stylesheets\bootstrap\_type.scss 28:19 @import
node_modules\bootstrap-sass\assets\stylesheets\_bootstrap.scss 18:9 @import
resources\assets\sass\frontend\frontend.scss 13:9 root stylesheet
: Using / for division is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($line-height-computed, 2)
More info and automated migrator: https://sass-lang.com/d/slash-div
╷
38 │ margin-top: ($line-height-computed / 2);
│ ^^^^^^^^^^^^^^^^^^^^^^^^^
╵
node_modules\bootstrap-sass\assets\stylesheets\bootstrap\_type.scss 38:16 @import
node_modules\bootstrap-sass\assets\stylesheets\_bootstrap.scss 18:9 @import
resources\assets\sass\frontend\frontend.scss 13:9 root stylesheet
: Using / for division is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($line-height-computed, 2)
I kindly ask you to replace division with math.div
See that: https://sass-lang.com/documentation/breaking-changes/slash-div
there is a tool to automate migration from / to math.div
I see that some months ago an user prepared this, but it's not ready to merge: #1218
I copy/paste from this link: #1218 (comment)
since this repo is no loger active maintains, i made a npm package use your pr, hope it is usefull
https://www.npmjs.com/package/bootstrap3-sass
https://github.com/kikyous/bootstrap3-sass
install:
npm install bootstrap3-sass
We resolved all issues