plotly/plotly.js

[BUG]: `@import` is deprecated

Opened this issue · 1 comments

jendrikw commented

Description

When using import "plotly.js/src/css/style.scss" in my code (using vite), I get a lot of deprecation warnings:

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

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

  ╷
1 │ @import "vars.scss";
  │         ^^^^^^^^^^^
  ╵
    ../../../plotly.js/src/css/style.scss 1:9  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

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

  ╷
4 │     @import "mixins.scss";
  │             ^^^^^^^^^^^^^
  ╵
    ../../../plotly.js/src/css/style.scss 4:13  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

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

  ╷
5 │     @import "base.scss";
  │             ^^^^^^^^^^^
  ╵
    ../../../plotly.js/src/css/style.scss 5:13  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

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

  ╷
6 │     @import "cursor.scss";
  │             ^^^^^^^^^^^^^
  ╵
    ../../../plotly.js/src/css/style.scss 6:13  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

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

  ╷
7 │     @import "modebar.scss";
  │             ^^^^^^^^^^^^^^
  ╵
    ../../../plotly.js/src/css/style.scss 7:13  root stylesheet

Steps to reproduce

https://stackblitz.com/edit/vitest-dev-vitest-k9muhcrx?file=package.json

Notes

Add info here that doesn't fit in the other sections.

camdecoster commented

Thanks for the bug report! This will get added to our backlog, but probably won't get any attention for a while. If you create a PR to update the SASS syntax, we could get someone to review it a lot sooner.