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.

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

Fix deprecation warning with sass 1.56.0 #37425

Merged
merged 1 commit into from
Nov 4, 2022
Merged

Conversation

blankse
Copy link
Contributor

@blankse blankse commented Nov 4, 2022

Closes #37424

Description

With the new sass version 1.56.0 (https://github.com/sass/dart-sass/releases/tag/1.56.0) I get following deprecation warning:

Deprecation $weight: Passing a number without unit % (100) is deprecated.

To preserve current behavior: $weight * 1%

More info: https://sass-lang.com/d/function-units

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would change existing functionality)

Checklist

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • My change introduces changes to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

@blankse blankse requested a review from a team as a code owner November 4, 2022 11:14
Copy link
Member

@julien-deramond julien-deramond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @blankse. We haven't bumped the dependency yet but it's gonna be useful when we'll do it :)

@AndriiTereshchuk
Copy link

AndriiTereshchuk commented Nov 21, 2022

Deprecation Warning: $weight: Passing a number without unit % (100) is deprecated.

To preserve current behavior: $weight * 1%

More info: https://sass-lang.com/d/function-units

201 │ @return mix(rgba($foreground, 1), $background, opacity($foreground) * 100);
│ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

node_modules/bootstrap/scss/_functions.scss 201:11 opaque()
node_modules/bootstrap/scss/mixins/_table-variants.scss 4:28 table-variant()
node_modules/bootstrap/scss/_tables.scss 147:3 @import
node_modules/bootstrap/scss/bootstrap.scss 20:9 @import
resources/sass/app.scss 5:9 root stylesheet

XhmikosR pushed a commit that referenced this pull request Nov 21, 2022
Deprecation Warning: $weight: Passing a number without unit % (100) is deprecated.
To preserve current behavior: $weight * 1%
More info: https://sass-lang.com/d/function-units
@Janatbek
Copy link

Janatbek commented Jan 6, 2023

I have sass 1.57.1, still getting same warning

@julien-deramond
Copy link
Member

Hey @Janatbek, if so please create a new issue with all the versions used to reproduce the problem.

@blankse
Copy link
Contributor Author

blankse commented Jan 6, 2023

@Janatbek Which bootstrap version do you use? This bugfix is only included in the current version v5.2.3

@moltar
Copy link

moltar commented Jan 8, 2023

I'm on 5.2.3 and I am still seeing this issue.

Deprecation Warning: $weight: Passing a number without unit % (80) is deprecated.

To preserve current behavior: $weight * 1%

More info: https://sass-lang.com/d/function-units

    ╷
207 │   @return mix(white, $color, $weight);
    │           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ╵
    node_modules/bootstrap/scss/_functions.scss 207:11      tint-color()
    src/website/web/src/theme/variables/palette.scss 26:15  @import
    src/website/web/src/theme/variables/index.scss 1:9      @import
    src/website/web/src/theme/theme.module.scss 7:9         @import
    src/website/web/src/theme/index.scss 3:9                root stylesheet
❯ npm ls bootstrap
[email protected]
└── [email protected]
    node_modules/bootstrap/scss/_functions.scss 207:11      tint-color()

⬇️

// scss-docs-start color-functions
// Tint a color: mix a color with white
@function tint-color($color, $weight) {
  @return mix(white, $color, $weight);
}

@moltar
Copy link

moltar commented Jan 8, 2023

Also, the notice on GH says:

The pull request first appeared in v5.3.0-alpha1

@XhmikosR
Copy link
Member

XhmikosR commented Jan 9, 2023

This was fixed in 5.2.3. If you are getting the warning, it's either your local modules or another part in your Sass.

https://github.com/twbs/bootstrap/actions/runs/3517040211/jobs/5894345876

@twbs twbs locked as resolved and limited conversation to collaborators Jan 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incompatible with sass 1.56.0
6 participants