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

Issue with ms-animation mixin with Sass (latest version of DartSass) #1275

Open
valentintudor opened this issue Sep 2, 2021 · 0 comments
Open

Comments

@valentintudor
Copy link

valentintudor commented Sep 2, 2021

There is an error with the ms-animation mixin from _Animation.Mixins.MDL2.scss which is also a deprecated file (imported in _Animation.Mixins.scss) and imported from _References.scss.

ModuleBuildError: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
SassError: $string: ms-fadeIn is not a string.

177 │ $namelist: append($namelist, unquote($newname), 'comma');

Some people recommend to use the latest node-sass version, but this is deprecated, since the dart-sass implementation of Sass with Dart language is now preferred. This would not be an option.

Please fix this issue, you can't build a Webpack project from Fluent UI Sass sources because of this small issue. These are deprecated stuff which should be removed in a future version, but currently it's blocking and you are required to use the CSS directly. I know this is a well known issue lasting since last year, but currently not fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant