-
Notifications
You must be signed in to change notification settings - Fork 40
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
;
are considered syntax error under sass
style tags
#4
Comments
Well, aren't ; syntax error in old sass? I'm not expert, only used less and scss (sass2). Look here: https://github.com/hedefalk/atom-vue/blob/master/grammars/vue.cson#L158 |
Sorry, Sass 3 that is. |
Well, yes, but:
(using webpack and vue-loader for compilation). |
I'm successfully using webpack with { test: /.scss$/, loader: "style!css!sass" }, in webpack.config.js and "sass-loader": "~2.0.1", in package.json -> devDependencies Oh, wait, maybe I'm not… I'm not actually using any styling in my components there, seems to be on the side… Please consider PR if you find any solution. This package was just something I auto-created for my own use mechanically (with a script) from the sublime one: https://github.com/vuejs/vue-syntax-highlight at a particular commit: vuejs/vue-syntax-highlight@4796727 Any work there by @yyx990803 et al hasn't been merged in here since. |
I'm not sure, but I would guess that https://github.com/vuejs/vue-loader simply delegates to a matching loader. So if you write lang=scss in your .vue to make this atom plugin happy, you need to have: { test: /.scss$/, loader: "style!css!sass" }, and not { test: /.sass$/, loader: "style!css!sass" }, in your webpack config. Can this be the issue? |
Please try the workaround @ShayDavidson |
less
works fine for exampleThe text was updated successfully, but these errors were encountered: