-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Configuration? #1
Comments
Ditto |
@garthk want push+publish access? |
@garthk : You cannot pass browsers in your plugin as requested by @tbusser. I commented and fixed your code. (https://gist.github.com/garthk/e468e708d1c3a4c06571) @balupton : When you want to implement it in your official plugin, make sure to use my fixes. (Btw, it is really easy - would be nice if the official plugin supported options.) |
@schlaup looks good, what does it do if you don't include any options by default? Might be worth specifying a sensible default, like 'last 2 versions, >5%' or something, if there isn't already a default (couldn't find any mention of a default in the core library). Do you fancy creating a pull request for this? |
The core has defaults. (They are a bit hidden in its sources.) So, if you do not specify a config, the plugin uses the defaults of autoprefixer. (['> 1%', 'last 2 versions', 'Firefox ESR', 'Opera 12.1']) I do not code in coffee script. Since it affects only a couple of lines, I hope @balupton can incorporate them without much trouble and without a pull request. In the meantime you can copy the files from @garthk with my fixes. Make sure the folder structure in your DocPad project is like this: plugins Run "npm install" in the folder and nothing else to do, DocPad automatically sees the plugin in the "plugins" folder. |
v2.1.0 released, still no custom config - happy for a PR |
Really this plugin should become just a postcss plugin |
Is there anyway to indicate which browser version should be supported like in the original project? For instance I need IE8 support but this is no longer covered by the default last two browser versions.
The text was updated successfully, but these errors were encountered: