-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Feature : Autoprefixer #85
Comments
vue-loader actually already does autoprefixing for component CSS: http://vuejs.github.io/vue-loader/features/postcss.html But yeah it doesn't get applied to non-component CSS, so I like your proposal, we can turn off vue-loader's own stuff and centralize it inside |
Ok I'll work on it tomorrow. I'll close this issue when the PR is submitted |
@Grafikart how is it going? Just stumbled on this problem myself. |
I did this feature for a project but I didn't add this to the template. Is there a way to test a local template @yyx990803 ? I don't want to make a broken PR :) |
@Grafikart I recommend forking the template, then you can run |
@Grafikart Just pinging on this. Anything we can do to help? |
No it's fine I just did the missing commits |
You can actually import an external stylesheet in your top-most (App) component. That gets parsed as well, right? One implementation to rule them all. |
I've added a new entry for other sass files that I don't want to go into a vue component directly. Unfortunately another sass entry doesn't get the autoprefixing. Is that something that can be added easily? I'm trying to figure this out for hours now. Adding sass directly to the webpack modules hash breaks everything unfortunately. |
That's what they're working on here. For now you could import it in your topmost component. That way you don't need another entry. |
@Grafikart Do you need any help on this? |
Autoprefixer is in the latest template release. This can be closed. |
Since this template is able to load CSS I was wondering if you would be interested into adding autoprefixer.
cssLoaders()
I can do a PR if you are interested.
The text was updated successfully, but these errors were encountered: