-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Sass deprecation warning: Using /
for division is deprecated
#34051
Comments
This comment has been minimized.
This comment has been minimized.
Same issue. Especially annoying since you can not "mute" sass warnings. This is a problem with Sass and not Bootstrap, but still affects the user experience. |
@ffoodd I just hit this in the npm starter project with sass 1.33.0. Now, it is a warning only, but it's annoying because it's repeated many times. My understanding is that the proposed methods don't work with node-sass which is OK for v5.0.0, but for v4 we need to think about it... I was planning to switch to sass in v4 too but perhaps we should do it in a minor version bump. PS. I run dependabot so that we get the updated PR and see the warnings. EDIT: https://github.com/twbs/bootstrap/pull/34057/checks?check_run_id=2636494233#step:6:12 |
/
for division is deprecated and will be removed in Dart Sass 2.0.0
This comment has been minimized.
This comment has been minimized.
Hope that the team will fix it soon! I am now unable to start dev build because of huge amount of warnings. Guys from sass refuse to add any silencer for warnings, because they think that warnings are critical, and you should never silence them, even if there are 100k warning lines in your dev build. UPDATE: Solved by manually downgrading sass to 1.32.* |
Can confirm downgrade works. Thanks @Onefivefournine |
This comment has been minimized.
This comment has been minimized.
Please everyone stop spamming the issue. We will find a solution for v5. You should bug though the Sass devs to allow to hide these warnings. IMHO it's pretty bad right now. |
sass/dart-sass has released v1.34.0 that adds a
|
- bootstrap v4 now throws deprecation warning with the newer releases of sass. They likely a won't fix until bootstrap v.5. See also: twbs/bootstrap#34051
This comment has been minimized.
This comment has been minimized.
@exeptionerror has been spamming the same link already 5 times or something. |
"solution"... by fixing sass compiler to an older version... a solution would be something like a polyfilling math.div() for node-sass and replaceing it everywhere in the code |
Polyfilling isn't an option IMHO, this is simply not compatible. AFAIK there's no way to either check for Any thoughts or ideas are welcome, obviously! |
Thanks, solved for Element Plus also. |
@yktoo repro with the sass warning logs similar to the original issue would be helpful. Also, make sure the warning isn't originating other dependencies you may be using (or your own sass files). After updating to |
@darrinmn9 Indeed, it was my own stylesheet that caused the warnings, after I replaced division with multiplication all warnings were gone. |
Adding the
|
* Fix frontend build * Update package-lock.json * Update package-lock.json * fix package-lock with npm install * downgrade sass to 1.32. Link to solution below: twbs/bootstrap#34051 * Update package-lock.json * Update package-lock.json * revert sass downgrade
It works for me |
[11:53:25] Starting 'build:compile-css'... Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div
298 │ $headings-margin-bottom: $spacer / 2 !default; Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($input-padding-y, 2) or calc($input-padding-y / 2) More info and automated migrator: https://sass-lang.com/d/slash-div
493 │ $input-height-inner-quarter: calc(#{$input-line-height * .25em} + #{$input-padding-y / 2}) !default; Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($custom-control-indicator-size, 2) or calc($custom-control-indicator-size / 2) More info and automated migrator: https://sass-lang.com/d/slash-div
560 │ $custom-switch-indicator-border-radius: $custom-control-indicator-size / 2 !default; Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div
706 │ $nav-divider-margin-y: $spacer / 2 !default; Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div
711 │ $navbar-padding-y: $spacer / 2 !default; [11:53:29] 'build:compile-css' errored after 3.58 s ╷ ╷ ╷ ╷ [11:53:29] 'build' errored after 9.13 s |
removed deprecated css warnings when uploading the application, using the concepts of the issue: twbs/bootstrap#34051
Removidos os avisos de CSS obsoletos, ao fazer o upload do aplicativo, a biblioteca sass apresentava conflitos com a biblioteca usada por eles do bootstrap foi corrigido usando os conceitos deste problema: twbs/bootstrap#34051
Removidos os avisos de CSS obsoletos, ao fazer o upload do aplicativo, a biblioteca sass apresentava conflitos com a biblioteca usada por eles do bootstrap foi corrigido usando os conceitos deste problema: twbs/bootstrap#34051
fixed sass compilation error when building the application, following the patterns used by the sass lib in the issue: twbs/bootstrap#34051
fixed sass compilation error when building the application, following the patterns used by the sass lib in the issue: twbs/bootstrap#34051 Added 'webpack' and 'cache-loader' dependencies so that the project can compile the '--quiet-deps' instruction in the 'serve' script
fixed sass compilation error when building the application, following the patterns used by the sass lib in the issue: twbs/bootstrap#34051 Added 'webpack' and 'cache-loader' dependencies so that the project can compile the '--quiet-deps' instruction in the 'serve' script
fixed sass compilation error when building the application, following the patterns used by the sass lib in the issue: twbs/bootstrap#34051 Added 'webpack' and 'cache-loader' dependencies so that the project can compile the '--quiet-deps' instruction in the 'serve' script
This fixes a large number of warnings during build time related to the [warning: Using / for division is deprecated][0] error [0]: twbs/bootstrap#34051
This fixes a large number of warnings during build time related to the [warning: Using / for division is deprecated][0] error [0]: twbs/bootstrap#34051
Because of twbs/bootstrap#34051, the sass package is currently set to "~1.32.11". Also see https://stackoverflow.com/questions/67675422/deprecation-warning-in-bootstrap-scss This serves as a reminder, so that we do not accidentally bump sass to the latest version for now, until this issue is resolved
UPDATE: After fixing the warnings in my own code, as if I were using bootstrap v5 with sass, I was checking that the latest version of sass that bootstrap uses so far is
|
dart-sass has deprecated
/
in favor ofmath.div
. Compiling https://github.com/twbs/bootstrap/blob/main/scss/_variables.scss now results with many sass warnings.The text was updated successfully, but these errors were encountered: