Disable Autoprefixer in cssnano

[ai]

Fix #281

[SpaceK33z]

@jhnns, do you think this change is fine? Apparently this is causing many many issues when using postcss.

[codecov-io]

Current coverage is 98.34% (diff: 100%)

Merging #361 into master will not change coverage

@@             master       #361   diff @@
==========================================
  Files             9          9          
  Lines           303        303          
  Methods          41         41          
  Messages          0          0          
  Branches         66         66          
==========================================
  Hits            298        298          
  Misses            5          5          
  Partials          0          0          

Powered by Codecov. Last update 22f6621...6f65e4b

[jhnns]

Yes, I think it's fine. I would consider this to be a bug, so this is not a major version bump. We (probably) won't break anything by adding prefixes, we will just make CSS files bigger (assuming that autoprefixer is working correctly 😉).

[jhnns]

And I assume that most developers were not aware of this (including me).

[ben-eb]

nb: autoprefixer in cssnano is removal-only, so it would have not added prefixes, only removed them. 😄

[ai]

Thanks. Very excited about this fix. Could you ping me on release, so I could tweet about this important fix for Autoprefixer users?

[jhnns]

nb: autoprefixer in cssnano is removal-only, so it would have not added prefixes, only removed them.

I know. But when we remove the autoprefixer from cssnano, we will essentially add prefixes to minified bundles :)

[ben-eb]

Ah yes, of course. 👍

[codeheroics]

Thanks for the great work! Is a release containing this fix soon planned for css-loader?

Thanks for releasing it :)