search

Aaike / jspm-less-plugin

systemjs plugin that allows injection of less files without the need for pre-compiling
MIT License
14 stars 9 forks source link

add autoprefixer #4

Open yordis opened 9 years ago

yordis commented 9 years ago

Hey I would like to see this https://github.com/less/less-plugin-autoprefix in the plugin by default. I think this is very useful tool that we all need so will be great if we can have it

micky2be commented 9 years ago

Absolutely not by default please

yordis commented 9 years ago

@micky2be can you please give arguments? Because I can't memorize all the compatibility status.

micky2be commented 9 years ago

Also, I might want to release on 1 browser only (via Cordova). Also I want to have control over what need to be prefixed or not. If everything is auto-prefixed it can becomes a nightmare. You barely need an auto-prefixer in my opinion, we are talking about Less here, if you need something make a mixin for it. Or use an existing one like http://lesselements.com/ So having it optional is good, but surely not as a default. Let me have the control of it.

yordis commented 9 years ago

Why nightmare? But it's ok at least let me use autoprefixer as a plugin

yordis commented 9 years ago

@Aaike are you maintaining this?

Aaike commented 9 years ago

Hey, i haven't looked at this in a while ... but please feel free to submit a PR if you want to add anything.

i would also suggest though to not automatically include the auto prefix. there should be a way for the user to change the configuration and plugins used.