allow to pass minimize options via query params

Author: sokra

README.md

@@ -219,7 +219,7 @@ This may change in the future, when the module system (i. e. webpack) supports l
 
 By default the css-loader minimizes the css if specified by the module system.
 
-In some cases the minification is destructive to the css, so you can provide some options to it. clean-css is used for minification and you find a [list of options here](https://github.com/jakubpawlowicz/clean-css#how-to-use-clean-css-programmatically). Just provide them as query parameter: i. e. `require("css-loader?-restructuring&compatibility")` to disable restructuring and enable compatibility mode.
+In some cases the minification is destructive to the css, so you can provide some options to it. cssnano is used for minification and you find a [list of options here](http://cssnano.co/options/). Just provide them as query parameter: i. e. `require("css-loader?-autoprefixer")` to disable removing of deprecated vendor prefixes.
 
 You can also disable or enforce minification with the `minimize` query parameter.
 

lib/processCss.js

@@ -124,7 +124,6 @@ module.exports = function processCss(inputSource, inputMap, options) {
 	var forceMinimize = query.minimize;
 	var minimize = typeof forceMinimize !== "undefined" ? !!forceMinimize : options.minimize;
 
-
 	var parserOptions = {
 		root: root,
 		mode: options.mode
@@ -159,12 +158,12 @@ module.exports = function processCss(inputSource, inputMap, options) {
 	]);
 
 	if(minimize) {
-		pipeline.use(cssnano({
-			zindex: false,
-			urls: false,
-			unused: false,
-			idents: false
-		}));
+		var minimizeOptions = Object.create(query);
+		["zindex", "urls", "unused", "idents"].forEach(function(name) {
+			if(typeof minimizeOptions[name] === "undefined")
+				minimizeOptions[name] = false;
+		});
+		pipeline.use(cssnano(minimizeOptions));
 	}
 
 	var result = pipeline.process(inputSource, {

test/moduleMinimizeTest.js

@@ -0,0 +1,17 @@
+/*globals describe */
+
+var test = require("./helpers").testSingleItem;
+
+var path = require("path");
+var fs = require("fs");
+var testCasesPath = path.join(__dirname, "moduleMinimizeTestCases");
+var testCases = fs.readdirSync(testCasesPath);
+
+describe("module", function() {
+	testCases.forEach(function(name) {
+		var source = fs.readFileSync(path.join(testCasesPath, name, "source.css"), "utf-8");
+		var expected = fs.readFileSync(path.join(testCasesPath, name, "expected.css"), "utf-8");
+
+		test(name, source, expected, "?module&minimize&sourceMap&-comments&localIdentName=_[local]_");
+	});
+});

test/moduleMinimizeTestCases/keyframes-and-animation/expected.css

@@ -0,0 +1,3 @@
+@keyframes _bounce_{0%{transform:translateY(-100%);opacity:0}5%{transform:translateY(-50%);opacity:1}}@keyframes _bounce2_{0%{transform:translateY(-100%);opacity:0}50%{transform:translateY(-50%);opacity:1}}
+
+/* comment */._bounce_{animation-name:_bounce_;animation:_bounce2_ 1s ease;z-index:1442}

test/moduleMinimizeTestCases/keyframes-and-animation/source.css

@@ -0,0 +1,28 @@
+@keyframes bounce {
+	0% {
+		transform: translateY(-100%);
+		opacity: 0;
+	}
+	5% {
+		transform: translateY(-50%);
+		opacity: 1;
+	}
+}
+
+@keyframes bounce2 {
+	0% {
+		transform: translateY(-100%);
+		opacity: 0;
+	}
+	50% {
+		transform: translateY(-50%);
+		opacity: 1;
+	}
+}
+
+/* comment */
+.bounce {
+	animation-name: bounce;
+	animation: bounce2 1s ease;
+	z-index: 1442;
+}