Problem/Motivation

There are some line on the components files that show the following deprecated warning:

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

╷
62 │ margin-top: $spacer / 2;
│ ^^^^^^^^^^^
╵
../../contrib/bootstrap_barrio/scss/components/commerce.scss 62:17 @import
../../contrib/bootstrap_barrio/scss/barrio.scss 8:9 @import
scss/import.scss 15:9 @import
scss/style.scss 7:9 root stylesheet

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

╷
99 │ padding: $spacer/2 $spacer;
│ ^^^^^^^^^
╵
../../contrib/bootstrap_barrio/scss/components/commerce.scss 99:14 @import
../../contrib/bootstrap_barrio/scss/barrio.scss 8:9 @import
scss/import.scss 15:9 @import
scss/style.scss 7:9 root stylesheet

Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.

Recommendation: math.div($spacer, 4) or calc($spacer / 4)

More info and automated migrator: https://sass-lang.com/d/slash-div

╷
110 │ padding: $spacer / 4;
│ ^^^^^^^^^^^
╵
../../contrib/bootstrap_barrio/scss/components/commerce.scss 110:14 @import
../../contrib/bootstrap_barrio/scss/barrio.scss 8:9 @import
scss/import.scss 15:9 @import
scss/style.scss 7:9 root stylesheet

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

╷
119 │ padding: $spacer/2 $spacer/4 !important;
│ ^^^^^^^^^
╵
../../contrib/bootstrap_barrio/scss/components/commerce.scss 119:16 @import
../../contrib/bootstrap_barrio/scss/barrio.scss 8:9 @import
scss/import.scss 15:9 @import
scss/style.scss 7:9 root stylesheet

Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.

Recommendation: math.div($spacer, 4) or calc($spacer / 4)

More info and automated migrator: https://sass-lang.com/d/slash-div

╷
119 │ padding: $spacer/2 $spacer/4 !important;
│ ^^^^^^^^^
╵
../../contrib/bootstrap_barrio/scss/components/commerce.scss 119:26 @import
../../contrib/bootstrap_barrio/scss/barrio.scss 8:9 @import
scss/import.scss 15:9 @import
scss/style.scss 7:9 root stylesheet

Warning: 9 repetitive deprecation warnings omitted.

CommentFileSizeAuthor
#5 patch-applied-cleanly.png69.55 KBanoopsingh92
#2 Deprecation_Warning_Using_for_division_outside_of_calc-3295840-2.patch3.11 KBmaxmendez

Issue fork bootstrap_barrio-3295840

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

  • 3295840-using--for Comparecompare
About issue forks

Comments

MaxMendez created an issue. See original summary.

maxmendez’s picture

maxmendez commented
StatusFileSize
new Deprecation_Warning_Using_for_division_outside_of_calc-3295840-2.patch3.11 KB

Here an approach

anoopsingh92 made their first commit to this issue’s fork.

anoopsingh92’s picture

anoopsingh92
Location Rajasthan, India
commented
Assigned: Unassigned » anoopsingh92
anoopsingh92’s picture

anoopsingh92
Location Rajasthan, India
commented
StatusFileSize
new patch-applied-cleanly.png69.55 KB

Hi @MaxMendez, I reviewed your patch #2. It is applied successfully and looks good to me.

Lenovo@LAPTOP-PDE747K8 MINGW64 /c/xampp/htdocs/drupal-9/web/modules/contrib/bootstrap_barrio-3295840 (3295840-using--for)
$ git apply -v Deprecation_Warning_Using_for_division_outside_of_calc-3295840-2.patch
Checking patch scss/components/commerce.scss...
Checking patch scss/components/site-footer.scss...
Checking patch scss/components/tabledrag.scss...
Checking patch scss/components/user.scss...
Applied patch scss/components/commerce.scss cleanly.
Applied patch scss/components/site-footer.scss cleanly.
Applied patch scss/components/tabledrag.scss cleanly.
Applied patch scss/components/user.scss cleanly.

Patch #2

Thanks for the patch @Max.

anoopsingh92’s picture

anoopsingh92
Location Rajasthan, India
commented
Assigned: anoopsingh92 » Unassigned
Status: Active » Needs review

  • hatuhay committed 662ba31 on 5.5.x authored by MaxMendez 
    Issue #3295840 by anoopsingh92, MaxMendez: Using / for division outside...
  • hatuhay committed c8415f0 on 5.5.x authored by MaxMendez
    Issue #3295840 by anoopsingh92, MaxMendez: Using / for division outside...

  • hatuhay committed e9a9ff0 on 5.x authored by MaxMendez 
    Issue #3295840 by anoopsingh92, MaxMendez: Using / for division outside...
hatuhay’s picture

hatuhay commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.